Использование strdup()

Question

Допустим у нас есть:

-структуру

struct Person { 
    char *name; 
}; 

struct Person *Person_create(char *name){ 
    struct Person *who = malloc(sizeof(struct Person)); 
    assert(who != NULL); 
    who->name = strdup(name); 
    return who; 
} 

void Person_destroy(struct Person *who){ 
    assert(who != NULL); 
    free(who->name); 
    free(who); 
} 

И главная функция:

int main(int argc,char *argv[]){ 

    struct Person *mike = Person_create("mike"); 
    Person_print(mike); 
    Person_destroy(mike); 

    return 0; 
} 

Приведенный выше код не будет работать должным образом без функции strdup(). Вальгринд говорит, что адрес, который вы пытаетесь освободить бесплатно (who-> name), не является malloc'd. Какая история за этим, разве я не помню эту память, когда я строил структуру? И какая разница делает strdup()?

Answer 1

В вашем коде каждый объект Person имеет два независимых блока памяти: объект struct Person и отдельный блок памяти для имени. Этот отдельный блок памяти указывается указателем name внутри структуры. Когда вы назначаете struct Person с помощью malloc, вы не выделяете никакой памяти для имени. Это, опять же, независимый блок памяти, который должен быть выделен независимо.

Как вы планируете распределять память для имени, полностью зависит от вас. Если вы используете strdup (который по существу является оберткой для malloc), как в вашем коде выше, тогда вам нужно будет освободить его на free в конце.

Вы не объяснили, что вы подразумеваете под «без функции strdup()». Как вы выглядели без кода? Если вы просто сделали

who->name = name; 

тогда вы сделали, что who->name указатель указывает непосредственно к буквальному памяти строки, занятой буквального "mike". Строковые литералы находятся в статической памяти. Вам не разрешено использовать free. Это то, что говорит вам Вальгринд.

Answer 2

strdup do noes call malloc, это только операция с строкой. вы только malloc указатель на структуру, а не внутренний член

Answer 3

Mallocing struct выделяет память для имени указателя, но не выделяет память для указания имени. В этот момент who-> name будет какой-то случайной мусорной ценностью, поэтому освобождение не имеет смысла.

strdup использует malloc для выделения памяти для строки, которую он копирует. После того, как вы вернете указатель из strdup, вы можете и должны освободить его, когда закончите.