0

我显然不是在这里责怪 printf，我可能搞砸了我的内存分配和访问，但我不明白我哪里做错了。程序在 main 中的第二个 printf 上崩溃。如果我评论第二个，它也会在第三个崩溃。实际上，每当我在第一个 printf 之后访问 p 时它就会崩溃！

有人可以解释我做错了什么吗？

非常感谢。

typedef struct 
{
    char * firstname;
    char * lastname;
    int age;
} person;

person * new_person(char * firstname, char * lastname, int age)
{
    person p;
    int lf = strlen(firstname);
    int ll = strlen(lastname);
    p.firstname = (char *)malloc(++lf * sizeof(char));
    p.lastname = (char *)malloc(++ll * sizeof(char));
    strcpy(p.firstname, firstname);
    strcpy(p.lastname, lastname);
    p.age = age;

    return &p;
}    

int main()
{
    person * p = new_person("firstname", "last", 28);

    printf("nom : %s ; prenom : %s ; age : %d\n", p->lastname, p->firstname, p->age);
    printf("nom : %s ; prenom : %s ; age : %d\n", p->lastname, p->firstname, p->age);

    printf("nom : %s ; prenom : %s ; age : %d\n", (*p).lastname, (*p).firstname,(*p).age);

    return 0;
}
4

4 回答 4

5

您正在返回一个局部变量的地址。

您可以修改您new_person的参数以获取参数（指向人员的指针），也可以malloc在函数内部进行操作并对其进行操作。

当函数返回时，您在函数中声明的人超出范围。之后发生的一切都是未定义的。它可能会巧合地保持其价值一段时间，但你不应该依赖于此。当你调用 printf 时，堆栈会增长并用新的东西覆盖你的人的旧位置。

于 2012-09-06T19:51:25.787 回答
3

我认为问题出在这一行：

return &p;

请注意，您正在返回一个指向局部变量的指针。这会导致未定义的行为，因为一旦函数返回，局部变量p就不再存在。结果，读取或写入该指针将读取或写入垃圾数据。

这不会立即崩溃的事实是编译器如何生成代码的产物。很有可能，您第一次调用 时printf，它会重用以前使用的空间，而p这种方式纯属巧合，效果很好。但是，在函数返回后，它的堆栈帧已经破坏了p. 结果，第二次调用printf是读取调用留下的垃圾数据printf，因此崩溃。

（特别是：当您传递参数时，它将指向字符串的指针复制到堆栈上，因此在printf运行时，它可能会丢弃原始指针，但使用副本。然后第二次调用从过期的printf堆栈帧中加载垃圾指针，因为它住在以前居住的同一个地址p。）

要解决此问题，请考虑更改p为指向 a 的指针person，然后使用malloc来分配它。这样，内存在函数调用之后仍然存在，所以这个崩溃应该会消失。

希望这可以帮助！

于 2012-09-06T19:52:21.540 回答
1 
 person p;

// stuff

return &p

这是错误的。函数返回后，局部变量将离开作用域——它的地址将无效。您必须在堆上分配结构：

person *new_person(char *firstname, char *lastname, int age)
{
    person *p = malloc(sizeof(*p));
    p->firstname = strdup(firstname);
    p->lastname = strdup(lastname);
    p->age = age;

    return p;
}
于 2012-09-06T19:52:38.280 回答
0

问题出在函数 new_person 中。您在堆栈上创建人员 p 并返回其地址。您需要分配 person* p = new person(.....

于 2012-09-06T19:52:35.823 回答