Valgrind 指示内存泄漏

Question

所以，我有一些代码，Valgrind 似乎表明存在内存泄漏，但我没有看到。有问题的代码是

void *run_client(void *n)
{

    char recv_buffer[1024];

    // This is random, but right now we will allocate 5120 bytes
    // for the data coming from the instrument.
    const int data_size = 5120;
    int numbytes;

    struct net_info *n_info;
    n_info = (struct net_info *)malloc(sizeof(struct net_info));
    n_info = (struct net_info *)n;

    int clientfd;
    clientfd = init_client(&n_info->cfg);

    while (!*(n_info->flag))
    {
        // Clear the buffer every time...
        memset(recv_buffer, 0, 1024);
        if ((numbytes = recv(clientfd, recv_buffer, data_size - 1, 0)) == -1)
        {
            perror("recv");
            exit(1);
        }
        if (recv_buffer[0] != '{' || recv_buffer[numbytes - 1] != '\n')
            continue;

        // remove last two bytes that are EOL indicators
        numbytes -= 2;
        recv_buffer[numbytes] = 0;

        wclear(n_info->packet_win);
        box(n_info->packet_win, 0, 0);

        mvwprintw(n_info->packet_win, 1, 1, "%s", recv_buffer);
        wrefresh(n_info->packet_win);
    }

    close(clientfd);
    free(n_info);
    return 0;
}

我不确定是否需要详细信息，但此处提供了这些信息，以便您了解上下文。上面的函数从 pthread_create 调用并在指定端口上侦听数据。然后将字符串数据发布到 ncurses window（代码是初步的，目前仅用于测试）。当主循环收到退出请求并且 while 循环中的标志设置为高时，函数退出。

来自 Valgrind 的消息是

==24037== 88 bytes in 1 blocks are definitely lost in loss record 14 of 55
==24037==    at 0x4C2FB0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==24037==    by 0x401897: run_client (ncurses-cpc.c:60)
==24037==    by 0x52906DA: start_thread (pthread_create.c:463)
==24037==    by 0x57D588E: clone (clone.S:95)

第 60 行指的是 n_info = (struct net_info *)malloc(sizeof(struct net_info));。如您所见，在退出函数之前请求与该结构关联的内存的 free 。我错过了什么吗？为什么 Valgrind 会标记这个？（当 valgrind 为运行且选项 leak-check=yes 时，消息发布在堆摘要消息之后。）

Answer 1

从评论看来，您希望将输入结构传递给客户端线程。泄漏的出现是因为您分配了一些内存并立即丢弃了指向它的指针。

不，你不能只做 -

struct net_info *n_info = n;

和free来自主循环的内存，因为那会导致释放后使用的bug。

您需要做的是将此缓冲区的所有权转移给线程。这基本上意味着从线程启动的那一刻起，输入缓冲区就归线程所有，并且线程有责任 free 它。

你需要去掉线程中的新malloc，直接使用指针即可。在线程使用完指针后，指针将变为 free'。

还要注意主循环（或任何其他线程）不应共享同一个缓冲区。这意味着主循环应该 malloc 为它创建的每个线程提供一个单独的缓冲区。

Answer 2

n_info = (struct net_info *)malloc(sizeof(struct net_info));

您刚刚分配了内存...以及下一行：

n_info = (struct net_info *)n;

您用 n 覆盖了 n_info，失去了对刚刚分配的内存的引用。函数末尾的 free() 实际上是在释放 n，而不是此 malloc() 分配的内存。

Valgrind 指示内存泄漏

Valgrind indicates a memory leak

c

memory

valgrind