使用动态内存分配返回指向结构的指针

Question

我正在尝试返回一个指向我创建的新结构的指针，但我遇到了分段错误并且不确定如何以正确的方式执行此操作。

这是我现在正在使用的：

 typedef struct person{
  int age;
  char *name;
 }Person;

 Person *new_person(int age, const char *name){
  Person *x = malloc(sizeof(x));
  x->age = age;
  strcpy(x->name, name);
  return x;
 }

我一直在使用我在网上找到的教程/其他问题中的代码进行调整。但我似乎无法弄清楚我做错了什么，无论是我分配内存的方式还是我返回指针的方式。

我遇到的另一个问题是将任何关联内存与某个人解除分配。假设我想删除一个人，并释放与该人动态关联的内存。我搜索了它，它说我应该使用free 方法。但是我仍然会遇到内存泄漏。我有这样的设置：

void kill_person(Person *x){
    free(x->name);
    free(x);
}

我应该打电话给free(*x)吗？我也应该打电话给free(x->name)吗？

Answer 1

Person *x = malloc(sizeof(x));

你的意思是写

Person *x = malloc(sizeof(*x));

---

sizeof x == sizeof(Person*)，所以你只为指针分配了足够的内存。你的下一个段错误将（可能）在这一行：

strcpy(x->name, name);

您从未分配过x->name。一个适当的功能是：

Person *new_person(int age, const char *name) {
    Person *x = malloc(sizeof *x);
    x->name = malloc(strlen(name) + 1);
    x->age = age;
    strcpy(x->name, name);
    return x;
} 

Answer 2

您需要为 Person 以及它的指针分配空间，即名称。否则，即使 Person 被分配，它的 name 部分也将保持未分配。

选项 1：

typedef struct person{
  int age;
  char *name;
 }Person;

Person *new_person(int age, const char *name){
  Person *x = (Person *) malloc(sizeof(Person));
  if (name != NULL) {
      x->name = (char *) malloc(strlen(name) + 1);
  }
  x->age = age;
  strcpy(x->name, name);
  return x;
 }
void kill_person(Person *x){
    free(x->name);
    free(x);
}
int main() {
  Person *p = new_person(28, "Neel Lohit");
  // code to use p, when done, call kill_person
  kill_person(p);
  return 0;
}

选项 2（具有固定静态分配的名称部分）：

#define MAX_SZ 100
typedef struct person{
  int age;
  char name[MAX_SZ+1];
 }Person;

Person *new_person(int age, const char *name){
  Person *x = (Person *) malloc(sizeof(Person));
  x->age = age;
  strcpy(x->name, name);
  return x;
 }
void kill_person(Person *x){
    free(x);
}
int main() {
  Person *p = new_person(28, "Neel Lohit");
  // code to use p, when done, call kill_person
  kill_person(p);
  return 0;
}