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标记此？ （当使用选项leak-check=yes运行valgrind时，该消息将在堆摘要消息之后发布。）

Answer 1

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

不，你不能只是-

struct net_info *n_info = n;

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

您需要做的就是将此缓冲区的所有权转移到线程中。 从根本上讲，这意味着从线程启动时开始，输入缓冲区归线程所有，并且线程有责任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()分配的内存。