c - 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 窗口(代码是初步的，目前仅用于测试)。当主循环收到退出请求并且 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;

并从主循环中释放内存，因为这会导致释放后使用的错误。

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

需要去掉线程中新的malloc，直接使用指针即可。在线程使用完指针后，指针将被释放。

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