C中的内存分配错误

Question

我已经在以下代码上工作了一段时间，当我尝试为结构及其元素分配/取消分配内存时出现了问题。感谢您对问题的任何见解。查看错误消息后，我认为问题出在以下事实：我试图释放一个我没有适当分配内存的元素，但是在查看代码时这对我来说并不明显。我还尝试了一些代码，在这些代码中我没有为结构的每个元素分别分配内存，但是效果并不理想。

typedef struct {
  char *cnet;
  char *email;
  char *fname;
  char *lname;
  char *tel;
} vcard;

vcard *vcard_new(char *cnet, char *email, char *fname, char *lname, char *tel)
{
  vcard* new = (vcard*)malloc(sizeof(vcard));
  printf("%lu\n", sizeof(new->tel) );
  new->cnet = malloc(sizeof(new->cnet));
  new->email = malloc(sizeof(new->email));
  new->fname = malloc(sizeof(new->fname));
  new->lname = malloc(sizeof(new->lname));
  new->tel = malloc(sizeof(new->tel));
  new->cnet = cnet;
  new->email = email;
  new->fname = fname;
  new->lname = lname;
  new->tel = tel;
  return new;
}

/* vcard_free : free vcard and the strings it points to
 */
void vcard_free(vcard *c)
{
  free(c->cnet);
  free(c->email);
  free(c->fname);
  free(c->lname);
  free(c->tel);
  free(c);
  return;
}

Answer 1

您的整个内存分配是错误的。这里有一些指针。

1. 您只为一个分配内存char *，这不是预期的。

   • 较少的内存分配，边界溢出的可能性。

2. 然后，通过将参数分配给将指针分配给已分配内存的相同变量来覆盖已分配内存。

   • 您最终导致内存泄漏。
3. 您尝试释放的指针不是malloc()and家庭返回的。
   • 尝试无效free()，导致未定义的行为
4. 您在中使用了错误的格式说明符 printf()
   • sizeof产生类型为的结果size_t，您必须使用它%zu来打印结果。

解决方案：

1. 分配足够的内存来存储预期的内容，例如预定义的大小

    #define CNETSIZ 32
    #define EMAILSIZ 64
    . . . . . 
    new->cnet = malloc(CNETSIZ);
    new->email = malloc(EMAILSIZ);
   

   或者，根据输入字符串的长度，例如

    new->cnet = malloc(strlen(cnet)+1);  //+1 for the space to null-terminator
   

2. 在vcard_new()函数内部，用于strcpy()从函数参数中复制内容，例如

   strcpy(new->cnet, cnet);
   strcpy(new->email, email);