Почему после очистки строки программа падает?

Question

[newmersedez]

Не могу понять, в чем заключается ошибка. Программа на данном этапе обрабатывает строки из файла. Дан файл, содержащий в себе инструкции вида:

prior=0 task='Task 1'
prior=0 task='Task 3'
prior=2 task='Task 4'
prior=2 task='Task 5'

Я считываю в переменную temp значение prior, а в строку msg_buf строку, содержащуюся в ' ';
Память под строку выделяю правильно, в конце строки добавляю '\0', но после выполнения функции free(msg_buf) программа падает.... Почему так происходит? Вот код:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
#include "message.h"

void    message_processing(FILE *file)
{
    int         i, j;
    int         temp = 0, size = 0;
    char        str_buf[BUFSIZ] = {0};
    char        *msg_buf = NULL;
    Message     message;

    while(!feof(file))
    {
        fgets(str_buf, sizeof(str_buf), file);
        strtok(str_buf, "\n");
        i = 6;
        while(isdigit(str_buf[i]))
        {
            temp = temp * 10 + (str_buf[i++] - '0');
        }
        i += 7;
        for(; i < strlen(str_buf) - 1; i++)
        {
            //printf("%c", str_buf[i]);
            msg_buf = (char *)realloc(msg_buf, ++size * sizeof(char) + 1);
            msg_buf[size - 1] = str_buf[i];   
        }
        msg_buf[size] = '\0';
        printf("%s\n", msg_buf);
        free(msg_buf);
    }
}

Comments

[Василий Банников]

А с каким сообщением падает то? Скорее всего-таки неправильно освобождение происходит

[newmersedez]

Василий Банников, печатает Task 1 и зависает

[newmersedez]

Василий Банников,
---------Opened "input2.txt" file---------
Task 1
Task 3
free(): double free detected in tcache 2
Aborted (core dumped)

[Василий Банников]

newmersedez, английском по белому написано "double free detected". Вы дважды освобождаете одну и ту же память

[newmersedez]

Василий Банников, Да, это я понимаю, но до меня никак не доходит, где ошибка :(

[Василий Банников]

newmersedez, попробуйте в отладчике запустить и пронаблюдать.
Для упрощения понимания - перенесите объявления переменных ближе к месту инициализации.

[Johnny Gat]

Василий Банников,

перенесите объявления переменных ближе к месту инициализации.

Если это чистый Си, то объявление всех переменных должно быть в начале функции.

[Roman]

newmersedez, https://en.cppreference.com/w/c/memory/realloc

Reallocates the given area of memory. It must be previously allocated by malloc(), calloc() or realloc() and not yet freed with a call to free or realloc. Otherwise, the results are undefined.

char *msg_buf = NULL;
UB

[Василий Банников]

Johnny Gat, ок. Не знал об этом

Answer 1

[Wataru]

Вы очищаете msg_buf в цикле же. На второй итерации у вас msg_buf уже очищен и вы его передаете в realloc и все падает.

Попробуйте после free присвоить msg_buf = NULL.