c - Malloc 无法为结构分配内存

Question

我遇到了Segmentation fault (core dumped) 错误。

ma​​in.c

#include "header1.h"

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

int main(int argc, char** argv) {
        struct t_list *list = NULL;
        doFill(list);
        printf("%s\n", list->name);
        free(list);
        return 0;
}

header1.h

#ifndef HEADER1_H
#define HEADER1_H

struct t_list {
  char *name;
  struct t_list *next;
};

void doFill(struct t_list *list);

#endif

worker.c

#include "header1.h"
#include <stdlib.h>

void doFill(struct t_list *list) {
    list = (struct t_list *) malloc(sizeof(struct t_list));
    char *tmp = "somename";
    list->name = tmp;
    list->next = NULL;
}

当我运行此 (gcc -g main.c worker.c -o test) 时，我得到(在 main.c 中的 printf 行):

Segmentation fault (core dumped)

在 gdb 我看到:

Temporary breakpoint 1, main (argc=1, argv=0x7fffffffddf8) at main.c:8
8       struct t_list *list = NULL;
(gdb) next
9       doFill(list);
(gdb) step
doFill (list=0x0) at worker.c:6
6       list = (struct t_list *) malloc(sizeof(struct t_list));
(gdb) p list
$1 = (struct t_list *) 0x0
(gdb) next
7       char *tmp = "somename";
(gdb) p list
$2 = (struct t_list *) 0x0

可以看到worker.c中的malloc没有为list变量分配内存(malloc前后的指针指向在 0x0)。

如果我将 doFill 过程中的代码移到 main.c 中，它会正常工作:

ma​​in.c

#include "header1.h"

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

int main(int argc, char** argv) {
    struct t_list *list;
    list = (struct t_list *) malloc(sizeof(struct t_list));
    char *tmp = "somename";
    list->name = tmp;
    list->next = NULL;
    printf("%s\n", list->name);
    free(list);
    return 0;
}

$ gcc -g main.c -o test
$ ./test
somename

这怎么可能？我做错了什么？

gcc version 4.8.2 (Ubuntu 4.8.2-19ubuntu1)

Answer 1

您没有收到 list 的新值。事实上，传入list是完全没有用的。最好传入此节点的 name。

typedef struct t_list List;

List *newListNode(char *name) {
    List *list = malloc(sizeof(*list));
    if (!list) return NULL;
    list->name = strdup(name);
    if (!list->name) { free(list); return NULL; }
    list->next = NULL;
    return list;
}

char *strdup(char *src) {   // if strdup doesn't already exist.
    char *dst = malloc(strlen(src) + 1);
    if (!dst) return NULL;
    strcpy(dst, src);
    return dst;
}

将节点添加到列表的前面:

List *listAdd(List *list, char *name) {
    List *newnode = newListNode(name);
    if (!newnode) return NULL;
    if (list) newnode->next = list;
    return newnode;
}

要删除列表，请记住删除 malloced 字符串:

void deleteList(List *list) {
  for (List *next; list; list = next) {
    next = list->next;
    free(list->name);
    free(list);
  }
}