strdup()结构中的内存泄漏

时间:2015-08-30 15:52:27

标签: c memory-leaks

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

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

struct Person *Person_create(char *name, int age, int height, int weight)
{
    struct Person *who = malloc(sizeof(struct Person));
    assert(who != NULL);

    who->name = strdup(name);
    who->age = age;
    who->height = height;
    who->weight = weight;

    return who;
}

void Person_destroy(struct Person *who)
{
    assert(who != NULL);

    /*free(who->name);*/
    free(who);
}

void Person_print(struct Person *who)
{
    printf("Name: %s\n", who->name);
    printf("\tAge: %d\n", who->age);
    printf("\tHeight: %d\n", who->height);
    printf("\tWeight: %d\n", who->weight);
}

int main(int argc, char *argv[])
{
    // make two people structures
    struct Person *joe = Person_create(
                                       "Joe Alex", 32, 64, 140);

    struct Person *frank = Person_create(
                                         "Frank Blank", 20, 72, 180);

    // print them out and where they are in memory
    printf("Joe is at memory location %p:\n", joe);
    Person_print(joe);

    printf("Frank is at memory location %p:\n", frank);
    Person_print(frank);

    // make everyone age 20 years and print them again
    joe->age += 20;
    joe->height -= 2;
    joe->weight += 40;
    Person_print(joe);

    frank->age += 20;
    frank->weight += 20;
    Person_print(frank);

    // destroy them both so we clean up
    Person_destroy(joe);
    Person_destroy(frank);

    return 0;
}

strdup()将返回Person_create结构中的名称。但是,我很困惑,如果它被free(who);释放,所有内存都应该被释放。但不是。如果它也被free(who->name);释放,则释放所有堆块。为什么free(who);不足以释放所有堆?

1 个答案:

答案 0 :(得分:2)

我认为你的答案的评论会处理你的问题,但我会更正式地回答。

这里有一条经验法则:拥有与mallocs一样多的释放。你错过了一个免费的。参考strdup()的文档:

  

strdup()函数应返回一个指向新字符串的指针,即   s1指向的字符串的副本。返回的指针可以   被传递给free()。如果是新字符串,则返回空指针   无法创建。

这不是我见过的最好的文档,但是“返回的指针可以传递给free()”的事实应该告诉你malloc是在引擎盖下使用的。

所以内存没有被释放的原因是因为你根本就没有释放它。你有两行malloc:

struct Person *who = malloc(sizeof(struct Person));

who->name = strdup(name);

进一步阐述:malloc做什么?它分配内存字节。假设指针是4个字节,字符是1个字节,而int是4个字节。

当你说

struct Person *who = malloc(sizeof(struct Person));

struct Person有一个指针(4个字节)和三个整数(12个字节)。您总共分配了16个字节。在此调用free将只释放那16个字节。

现在说我们有这条线:

who->name = strdup("Bob");

strdup(“Bob”)使用带有3个字符的malloc创建一个新的char *,因为/0而总共有4个字节。

总共,为结构分配了16个字节,从strdup()分配了4个字节,但在“who”上调用free只释放了16个字节。你仍然有strdup()剩下的4个字节。