Не удается освободить только один динамически размещенный элемент - PullRequest
0 голосов
/ 16 мая 2019

Так что в основном у меня есть функция, которая определяет структуру университета и заполняет ее студентами. Все работает! Когда университет возвращается к основному, все значения сохраняются в главном университете.
Затем, когда я пытаюсь освободить его (в основном), по какой-то причине он глючит на braude.students[2].name и больше ни на кого.
Исключением является: CtrlsValidHeapPointer (block).
Почему он освобождает все другие динамически назначаемые имена, но не это?

university getStudentInfo(FILE * file) // Definition of the getStudentInfo function
{ // This function reads student information from a file
  // and updates a university with this information using a pointer
    int i = 1, j;
    char name[99]; // A temporary name to hold the student's name.
    university tempUni; // A temporary university to hold values
    Stud * temp = (Stud*)malloc(sizeof(Stud)); // Allocating memory
                                              //  for an array of students

    if (temp == NULL) // In case there wasn't enough space 
        Error_Msg("Couldn't allocate enough memory.");

    while (getInfo(file, name, &temp[i - 1]) == 8)
    // Using getInfo==8, because in each row, there are 8 variables to scan
    {
        temp[i - 1].name = (char*)malloc(sizeof(char)*strlen(name) + 1);
        // Allocating memory for the current element's name.

        if (temp[i - 1].name == NULL) // In case there wasn't enough space, terminate
            Error_Msg("Couldn't allocate enough memory.");
        strcpy(temp[i - 1].name, name);
        // If there was, copy the name from "name" to the current student's name.

        i++; // Increase i by one to have space in memory for one more student
        temp = (Stud*)realloc(temp, i * sizeof(Stud));
        // Realloc temp, to make more space for one more student.

        if (temp == NULL) // In case there wasn't enough space, terminate.
            Error_Msg("Couldn't allocate enough memory.");
    }

    tempUni.students = (Stud*)malloc(sizeof(temp));
    // Allocating memory for students of the university, with the size of temp

    if (tempUni.students == NULL)
        // In case there wasn't enough space, terminate.
        Error_Msg("Couldn't allocate enough memory.");

    tempUni.students->marks[5] = '\0'; // Making the last mark in the string \0
    tempUni.students = temp; // Let the temporary university array of students be temp 
    tempUni.studentCount = i - 1; // How many students
    return tempUni; // Update the pointed university to have the same values as tempUni
} 

Но когда я освобождаю динамически выделенную память в основном, вот так:

free(braude.students[0].name);
    free(braude.students[1].name);
            free(braude.students[2].name); // Crashes here???
            free(braude.students[3].name);
            free(braude.students);

1 Ответ

0 голосов
/ 16 мая 2019

Эта строка

tempUni.students = (Stud*)malloc(sizeof(temp));

выделяет память только для одного указателя.Код не так легко следовать, но, возможно, это должно быть

tempUni.students = malloc(i * sizeof(Stud));
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...