问题描述
我遇到了问题,我真的不知道为什么会这样以及如何发生。我会尽量解释自己。
代码有两个功能:main 和 assemble。在 main 中,我只是获取一些文档文件方向的参数,然后将每个参数发送到 assemble 函数。
在assemble函数中我打开文件,然后我开始一个链表,我为第一个节点定位内存,然后我在列表中添加了很多节点(那部分我没有在这里复制,因为它很大但它工作正常)然后最后我关闭文件并释放链表。
这是代码:
typedef struct node {
int location;
char name[MAX_LABEL];
char type;
struct node * next;
} labVar;
int main(int argc,char *argv[]) {
if (argc == 1)
printf("No file attached./n");
else {
while(--argc) {
argv++;
assemble(*argv);
}
}
return 0;
}
void assemble(char *file) {
FILE *iofp;
int i = 0;
labVar *nodeLabHead = NULL; /* initializing head of the labels' linked list */
labVar *nodeLabCurr = NULL; /* variable to go through the list */
if ((iofp=fopen(file,"r")) == NULL)
printf("\nCan't open file \"%s\"\n",file);
else {
printf("\nSuccess opening %s\n",file);
nodeLabHead = (labVar *) malloc(sizeof(labVar));
if (nodeLabHead == NULL) {
printf("\nNo memory to allocate.\n");
exit(0);
}
/*HERE I FILL THE LINKED LIST WITH MANY NODES IN ANOTHER FUNCTION,IT WORKS*/
fclose(iofp);
while (nodeLabHead->next != NULL) {
nodeLabCurr = nodeLabHead;
nodeLabHead = nodeLabHead->next;
free(nodeLabCurr);
}
free(nodeLabHead);
}
}
因此,对于我作为参数发送的所有类型的文件,代码似乎都可以正常工作。但是,当我尝试使用多个参数运行代码(为每个参数调用 append 函数)时,它会进入无限循环。我知道这与释放内存有关,因为它卡在了那个循环中。如果删除最后一行 (free(nodeLabHead);),它只会在我输入具有更多节点的文件然后输入更少节点的文件时卡住,否则它会起作用。
我还注意到,一开始,当将内存分配给第一个节点时,nodeLabHead->next==NULL 会按预期返回 1,但是在第二次它到达那里时(使用第二个文件,nodeLabHead ->next==NULL 返回 0,这意味着它没有真正被释放,我不明白为什么。
解决方法
您没有显示您的代码,但听起来发生的事情是您从未将 nodeLabHead->next 初始化为 NULL,因此它包含分配的内存中发生的任何事情。碰巧的是,您第一次调用函数时,它是 0,但后来不是。
您从 malloc
取回的内存未初始化——因此可能包含任何内容。如果您希望它具有可预测的值,则需要显式设置结构的每个字段。
尝试将 malloc
调用更改为 calloc
,或者在 memset(nodeLabHead,sizeof(labVar))
调用之后添加一个 malloc
。