簡體   English   中英

Valgrind指示內存泄漏

[英]Valgrind indicates a memory leak

因此,我有一些代碼,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時,該消息將堆摘要消息之后發布。)

從注釋看來,您想要將輸入結構傳遞給客戶端線程。 出現泄漏是因為您分配了一些內存並立即丟棄了指向它的指針。

不,你不能只是-

struct net_info *n_info = n;

並從主循環中free內存,因為這將導致釋放錯誤后使用。

您需要做的就是將此緩沖區的所有權轉移到線程中。 從根本上講,這意味着從線程啟動時開始,輸入緩沖區歸線程所有,並且線程有責任free它。

您需要在線程中刪除新的malloc ,然后直接使用指針。 線程使用完之后,指針將被free

還應注意主循環(或任何其他線程)不應共享同一緩沖區。 這意味着主循環應該為它創建的每個線程malloc一個單獨的緩沖區。

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

您剛剛分配了內存...以及下一行:

n_info = (struct net_info *)n;

您用n覆蓋n_info ,從而丟失了對剛剛分配的內存的引用。 函數末尾的free()實際上是釋放n ,而不是釋放此malloc()分配的內存。

暫無
暫無

聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.

 
粵ICP備18138465號  © 2020-2024 STACKOOM.COM