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_info覆盖了n，丢失了对刚刚分配的内存的引用。该功能结尾的free()实际上正在释放n，而不是释放该malloc()分配的内存。

Valgrind指示内存泄漏

2 个答案: