von malloc und free Verhalten verwirrt

Question

Ich bin verwirrt mit dem Einsatz von free() in Bezug auf den Datenstrukturen in C.

Wenn ich die folgenden Datenstruktur:

struct Person { 
    char *name; 
    int age; 
    float weight; 
    float height; 
}; 

ich für die Speicher am Zuteilen Struktur über malloc() ebenfalls: struct Person *me = malloc(sizeof(struct Person));

Nachdem ich meine Struktur mit der Verwendung fertig bin (rechts vor dem beenden des Programms), ich gehe zugewiesen um den Speicher zu befreien, wie folgt aus:

free(person_pointer->name); 
free(person_pointer); 

die Speicher Befreit, dass die name Zeiger auf mein Wissen notwendig sind, denn wenn ich nur frei person_pointerich nur den Speicher frei, die die Datenstruktur und ihre Mitgliederaber nicht der Speicher zum Speichern zugewiesen wurden Darauf verweisen Zeiger - Mitglieder der Struktur.

Allerdings scheint mit meiner Implementierung valgrind darauf hinzuweisen, dass die erste free() ungültig ist.

Also meine Frage läuft auf: Wenn ich den Speicher freigeben, den eine Struktur über einen Zeiger einnimmt, sollte ich präventiv den Speicher freigeben, auf den Mitgliedszeiger zeigen oder nicht?

EDIT: Das ist mein ganzer Code:

#include <stdio.h> 
#include <stdlib.h> 

struct Person { 
    char *name; 
    int age; 
    float weight; 
    float height; 
}; 

int main(int argc, char **argv) 
{ 
    struct Person *me = malloc(sizeof(struct Person)); 
    me->name = "Fotis"; 
    me->age = 20; 
    me->height = 1.75; 
    me->weight = 75; 

    printf("My name is %s and I am %d years old.\n", me->name, me->age); 
    printf("I am %f meters tall and I weight %f kgs\n", me->height, me->weight); 

    free(me->name); 
    free(me); 

    return 0; 
} 

Answer 1

me->name = "Fotis"; 

/* ... */ 

free(me->name); 

Die Regel lautet:

1 malloc = 1 free 

Sie nicht malloc auf me->name verwendet haben, so dass Sie nicht haben free es.

BTW, me->name sollte von const char * Typ sein.

Answer 2

Wenn Sie das tun

me->name = "Fotis"; 

Der name Zeiger nicht von malloc alloced wird, verweist er auf einen Stapel Variable, die in der Strings Tabelle Ihrer Binärdatei bei der Kompilierung gespeichert, daher kann man nicht frei es.

Die Faustregel lautet: Nur frei, was Sie malloced haben.

Sie können diese schreibgeschützte Zeichenfolge jedoch nicht aktualisieren.

Wenn ja so etwas wie:

me->name = strdup("Fotis"); 

Da strdup eine malloc tut (siehe Handbuch), haben Sie es zu befreien, und Sie können die Zeichenfolge nach seiner Erstellung aktualisieren.

Answer 3

Ja, Sie müssen den gesamten Speicher von Zeigern innerhalb der Struktur freigeben, wenn Sie sie zugewiesen haben.

Stellen Sie außerdem sicher, dass Sie die Mitglieder freigeben, bevor Sie die Struktur freigeben.

Eine einfache Art sich zu erinnern ist, sie in der umgekehrten Reihenfolge freizugeben, in der Sie zugewiesen haben.

Answer 4

Das Problem ist, dass Sie nicht wirklich den Char * Namen in Ihrer Struktur malloc'ed.

struct Person *me = malloc(sizeof(struct Person)); 
me->name = strdup("Fotis"); 
... 
free(me->name); 
free(me); 
return (0); 

Wenn Sie schreiben diese

me->name = "Fotis"; 

Sie eigentlich gar nicht malloc, der Zeiger Name weist auf einen Stapel Variable vom Typ const char *, wich nicht malloc'ed.