Функция освобождения структуры в C

Question

У меня проблема, когда, когда я вызываю функцию, которую я написал для бесплатной структуры, моя программа завершается первой свободной инструкцией.

Вот функция:

void deleteJobNode(struct jobNode *node) { 
    free(node->burstTime); 
    free(node->jobIndex); 
    free(node->next); 
    free(node->priority); 
    free(node->remainingTime); 
    free(node); 
} 

Вот определение структуры и функция, используемая для выделения памяти для структуры.

struct jobNode { 
    struct jobNode *next; 
    int jobIndex; 
    int burstTime; 
    int remainingTime; 
    int priority; 
}; 

struct jobNode *createJobNode(int newJobIndex, int newBurstTime, int newRemainingTime, int newPriority) { 
    struct jobNode *newNode = (struct jobNode *)malloc(sizeof(struct jobNode)); 
    newNode->jobIndex = newJobIndex; 
    newNode->burstTime = newBurstTime; 
    newNode->remainingTime = newRemainingTime; 
    newNode->priority = newPriority; 

    if(newNode == NULL) 
     printf("Node Creation Error"); 

    return newNode; 
} 

Я редактировал свободную функцию, как предложено:

void deleteJobNode(struct jobNode *node) { 
    free(node); 
} 

Однако я получаю эту ошибку во время выполнения:

*** Error in `/home/jack7521/workspace/req1b/Debug/req1b': double free or corruption (fasttop): 0x00000000010c2090 *** 

Answer 1

Вы делаете один вызов к таНос; так почему вы думаете, что вам нужно несколько звонки на бесплатно?!

Вещь: вам нужно только бесплатно эта структура один раз - окончательный free(node) выполняет эту работу!

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

Или под другим углом: malloc дает вам кучу байтов. Тот факт, что вы позже используете эти байты, используя этот «struct» «overlay», не важен вообще!

Answer 2

Для каждого malloc вам нужен free. Вам не нужно освобождать нединамические поля struct, они уже являются частью самого struct, поэтому достаточно освободить указатель на все struct.

Вам просто нужно:

free(node); 

Answer 3

Вы только должны использовать free с указателем на экземпляр структуры.

void deleteJobNode(struct jobNode *node) { 
    free(node); // will do the whole job 
} 

Потому что, когда вы выделить память с malloc вызова это один большой пучок памяти, которая может быть выпущена сразу с free.

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

Answer 4

Вы возвращаете адрес локальной переменной (newNode) в функцию createJobNode. newNode не имеет видимости вне функции createJobNode.Попробуйте выделить память для newNode в основной функции и передать его в качестве параметра в createJobNode (в данном случае это будет функция пустоты) как этого

void createJobNode(struct jobNode* newNode, ...) 
{ 
    //Fill the fields of newNode 

} 

int main(void) 
{ 
    struct jobNode * newNode = malloc(sizeof(struct newNode)); 

    createJobNode(newNode, ...); 

} 

Чтобы освободить выделенную переменную newNode, вам нужно звоните по телефону:

free(newNode);