Question

我正在尝试创建一个从.txt文件加载数据的函数，但是当它运行时，我总是会遇到分段错误（核心转储）错误。该文件包含未知数量的行，而每行都有一个字符串和一个由制表符分隔的整数。list_create函数仅创建数据结构。最后的while循环删除了数据结构，我没有包含代码，因为我确定它不会导致问题，但是我也想表明我正在释放数据结构。值得一提的是，何时使用gdb，我得到：

Program received signal SIGSEGV, Segmentation fault.
0x0000555555554c46 in load (filename=0x7fffffffe2ab "students.txt", 
    l=0x555555757260) at Student.c:92
92                  tmp->next=malloc(sizeof(struct _node));

我试图用其他方法来改变feof，在有无错误的情况下使用，并将fopen的模式更改为r而不是a。

#include <stdio.h>
#include <stdlib.h>
#include <assert.h>
#include <string.h>
#define MAXSTRING 50
typedef struct{
    char name[MAXSTRING];
    int id;
} student;
typedef struct _node* node;
typedef struct _list* list;

struct _node {
    student data;
    node next;
};

struct _list {
    node head;
    int size;
};


list list_create(){
    list l=(list) malloc(sizeof(struct _list));
    assert(1);
    l->head=NULL;
    l->size=0;
    return l;
}
void load(char*filename,list l){
    FILE *fd=fopen(filename,"r");
    node tmp=l->head;

    if(fd==NULL){
        printf("Error trying to open the file\n");
                abort();
    }
    else{

                while(!feof(fd)&&!ferror(fd)){

        fscanf(fd,"%s\t%d\n",tmp->data.name,&tmp->data.id);   
                tmp->next=(node)malloc(sizeof(struct _node));
        assert(tmp->next);
        tmp=tmp->next;                
        l->size++;
            if (tmp==NULL){
                printf("Error trying to allocate memory\n");
                abort();
            }
                }      
        }

    tmp->next=NULL;
    fclose(fd);
}   



int main(int argc,char *argv[]){ 
list l=list_create();
if(argc!=2){
        printf("Input Error\n");
    }
    load(argv[1],l);
\*Some code*\


while (!list_empty(l)){
                list_freenode(list_deletefirst(l));
        }

    free(l);
        return 0;

我希望能够成功加载文件，能够编辑其组件并保存它们。

Answer 1

在list_create中，您要做

l->head=NULL;

然后在load中完成

node tmp=l->head;

所以现在tmp为NULL。

但是稍后在load中，您将取消引用tmp（例如，在这里tmp->next= ....）。因此，您取消引用NULL指针会导致程序崩溃。

加载文件时出现分段错误（核心转储）

1 个答案: