Funktion zum Freigeben einer Struktur in C

Question

Ich habe ein Problem, wo, wenn ich eine Funktion, die ich schrieb, um eine Struktur freizugeben mein Programm auf der ersten freien Anweisung beendet.

Hier ist die Funktion:

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

Hier ist die Definition der Struktur und die Funktion benutzten Speicher zum struct zuzuteilen.

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; 
} 

Ich habe die freie Funktion wie vorgeschlagen bearbeitet:

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

Jedoch habe ich diesen Fehler zur Laufzeit erhalten:

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

Answer 1

Sie tun ein Aufruf malloc; also warum denkst du, du brauchst mehrere Anrufe an frei?!

Thing ist: Sie müssen nur kostenlos diese Struktur einmal - die endgültige free(node) macht den Job!

Denken Sie daran, was diese Methoden für Sie tun: malloc markiert einen zusammenhängenden Speicherbereich für den Einsatz; und frei gibt Bereich. Es hat keinen Sinn, die Mitglieder Ihrer Struktur überhaupt freizugeben. Denn sie sind innerhalb dieser Bereich sowieso.

Oder aus einem anderen Blickwinkel: malloc gibt Ihnen eine Menge Bytes. Die Tatsache, dass Sie später diese Bytes mit diesem "struct" "overlay" verwenden, ist überhaupt nicht wichtig!

Answer 2

Sie benötigen einen free für jeden malloc. Sie müssen nicht nicht dynamische Felder eines struct freigeben, sie sind bereits Teil des struct selbst, so dass der Zeiger auf das gesamte struct frei ist.

Sie brauchen nur:

free(node); 

Answer 3

Sie nur free mit dem Zeiger auf die Struktur-Instanz verwenden.

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

Denn wenn man etwas Speicher mit einem malloc Anruf zuweisen diese ein großes Bündel Speicher ist, der dann mit free sofort freigegeben werden.

Nur wenn Ihre Struktur Zeiger enthält, die selbst auf einen anderen dynamisch zugewiesenen Speicher zeigen, müssen Sie free mit diesen Strukturelementen aufrufen.

Answer 4

Sie geben die Adresse einer lokalen Variablen (newNode) in der Funktion 'createJobNode' zurück. newNode hat keine Sichtbarkeit außerhalb der Funktion 'createJobNode'.Versuchen Sie Speicher für die newNode in Ihre Hauptfunktion zuweisen und geben sie als Parameter an die createJobNode wie diese

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

} 

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

    createJobNode(newNode, ...); 

} 

(in diesem Fall wäre es eine Lücke Funktion sein) Um die zugewiesene Variable newNode zu befreien, müssen Sie Anruf:

free(newNode);