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

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

Вы только что выделили память ... и следующую строку:

n_info = (struct net_info *)n;

Вы перезаписываете n_info с помощью n, теряя ссылку на только что выделенную память. free() к концу функции фактически освобождает n, а не память, выделенную malloc().

Answer 2

Из комментариев видно, что вы хотите передать входную структуру клиентскому потоку. Утечка возникает потому, что вы выделяете некоторую память и сразу отбрасываете указатель на нее.

Нет, вы не можете просто сделать -

struct net_info *n_info = n;

и free память из основного цикла, потому что это приведет к использованию после свободной ошибки.

Что вам нужно сделать, это передать владение этим буфером потоку. Это в основном означает, что с момента запуска потока входной буфер принадлежит потоку, и поток несет ответственность за free его.

Вам нужно удалить новый malloc в потоке и просто использовать указатель напрямую. Указатель будет free 'd после того, как поток завершит его использование.

Также будьте осторожны, чтобы основной цикл (или любой другой поток) не разделял один и тот же буфер. Это означает, что основной цикл должен malloc создавать отдельный буфер для каждого потока, который он создает.