请大家看一下，这段代码不会出现内存泄露吧

xiao1tong1

请大家看一下，这段代码不会出现内存泄露吧
        int len = 10;
        char name[] = "1234567890"
        struct entry_t{
                char *name;
                int len; //姓名长度
        } *my_entry;
       
        my_entry = (struct entry_t *)malloc(sizof(entry_t) + len + 1);
        my_entry->name = my_entry + sizeof(entry_t);
        my_entry->len = len;
        strcpy(my_entry->name, name);
        ……
        free(my_entry);

cookis

不会.. 不过要注意
1. my_entry->name[len] = '\0'
2. strncpy (my_entry->name, name, len);

Godbach

应该没问题

ytl

原帖由 xiao1tong1 于 2008-9-10 11:28 发表
请大家看一下，这段代码不会出现内存泄露吧
        int len = 10;
        char name[] = "1234567890"
        struct entry_t{
                char *name;
                int len; //姓名长度
        } *my_entry;
       
        my_entry = (struct entry_t *)mall ...

int len = 10;         char name[] = "1234567890"         struct entry_t{                 int len; //姓名长度                 char name[];         } *my_entry;                  my_entry = (struct entry_t *)malloc(sizof(entry_t) + len + 1);         my_entry->len = len;         strncpy(my_entry->name, name, len);         ……         free(my_entry);

nicozhou

谁保证malloc就一定申请成功？写代码严谨些好

emacsnw

原帖由 xiao1tong1 于 2008-9-9 19:28 发表
请大家看一下，这段代码不会出现内存泄露吧
        int len = 10;
        char name[] = "1234567890"
        struct entry_t{
                char *name;
                int len; //姓名长度
        } *my_entry;
       
        my_entry = (struct entry_t *)mall ...

这样是可以的，还见过 0 长数组：

struct entry_t {
  int len;
  char name[0];
};
这样malloc了足够内存后可以省掉对name的赋值。

xiaonanln

        int len = 10;  
        char name[] = "1234567890"
        struct entry_t{
                char *name;
                int len; //姓名长度
        } *my_entry;
        
        my_entry = (struct entry_t *)malloc(sizof(entry_t) + len + 1); // 需要判断malloc成功（使用int len=10，不如直接使用sizeof(name) ）
        my_entry->name = my_entry + sizeof(entry_t); //错误！！！应该是 (char *)(my_entry + 1)
        my_entry->len = len;
        strcpy(my_entry->name, name);     //应该使用strncpy
        ……
        free(my_entry);  // 这里不会内存泄漏

程序里由一个错误

emacsnw

原帖由 xiaonanln 于 2008-9-9 20:48 发表

程序里由一个错误

楼主是在考大家吗？呵呵。
长度10不够吧。