【发布时间】:2013-02-20 06:24:01
【问题描述】:
我正在为我的班级用 C 语言编写程序,并且我正在做结对编程。我想知道它是如何不使程序崩溃的。我的伙伴在下面添加了这段代码,我运行 valgrind 来测试是否有任何内存泄漏,这似乎很好 - 再说一次,我是新手。无论如何,我想知道这段代码是如何工作的,因为当我释放 curr 时,接下来运行的是 curr = curr->link - 但我们只是释放了 curr!那么 curr 被取出内存怎么会有链接呢?我认为这样的事情是行不通的。
这是我的搭档写的:
for(node curr = list->head; curr != NULL; curr = curr->link ){
if (!dflag) printf("%s\n",curr->words);
else printf("[%p]\n",curr);
free(curr->words);
free(curr);
}
如果 ^ 这个 ^ 崩溃了,我正在考虑这样做:
for (node curr = list->head; curr != NULL; ) {
if (!dflag) printf("%s\n",curr->words);
else printf("[%p]\n",curr);
node prev = curr;
curr = curr->link;
free(prev);
}
另外,这是我们制作节点的方法:
typedef struct node *node;
struct node {
cstring words;
node link;
};
【问题讨论】:
-
这是一个非常糟糕的主意:
typedef struct node *node;。现在struct node和node都是名称类型,但它们命名不同的类型。埃加兹! -
未定义的行为不会导致崩溃。访问已释放的指针会导致未定义的行为。
-
问题很可能隐藏在
typedef struct node *node中。第一个 sn-p 显然是错误的,因为它在free(curr)之后取消引用curr。 -
释放内存时,“free”函数不会重置指针。我们必须在释放内存后显式初始化指针为 NULL。