意外的strcpy段错误

Question

我发现一个segfault无法解析。 免得您认为我没有搜索过，我认为问题与该问题相同。 我有以下类型定义的结构：

typedef struct usage usage;
struct usage{
      char name[9]; 
      int loc;  
      usage *next;
};

我正在从一个文件中读取数据，该文件由数字K和其后的K对（S，D）组成，其中S = 8个字符的字符串[这是变量名]和d =整数[存储位置]。

这是导致错误的代码：

void addUse(int index,char *nm, int addr){
     usage *temp;
     strcpy(temp->name,nm); //segfault here. 
     temp->loc = addr;
     temp->next= NULL;
    /* more processing */
}

为了使这一点更清楚，我从一个我有

int dummyIndex = 1;
char s1[9];
int val1; 
scanf(" %s %d, s1, &val1);
addUse(dummyIndex, s1, val1);

似乎在我链接到该问题的问题中，它们没有在堆上分配char 。 我不确定这里发生了什么。 在具有字段char name[9]另一个struct上对strcpy使用相同的调用就可以了。

我想念什么？ 我看了什么？

提前致谢！

Answer 1

您忘记了初始化临时指针，因此它指向的是随机内存。 当你写的时候

strcpy(temp->name, mm);

您正在跟踪一个指向随机地址的指针，并在其中写入字节，因此出现了段错误。

Answer 2

问题出在代码的以下行中

usage *temp;
strcpy(temp->name,nm); //segfault here. 

您没有初始化temp指针，因此它正在获取垃圾值。

尝试在strcpy之前对其进行初始化

usage *temp = malloc(sizeof(usage));

希望这可以帮助。

Answer 3

这是我的问题：

在其他调用中，我确保设置temp = malloc(sizeof(struct)); 不在这其中，所以我试图写入尚未分配的内存！

Answer 4

代码的问题很明显，并且SegFault发生在应有的确切位置。 原因是您正在使用以前未分配的内存。

void addUse(int index,char *nm, int addr){
     usage *temp;
     strcpy(temp->name,nm); //segfault here. 
     // ...
}

当您声明“ 用法 * temp;”时 您正在创建使用类型的指针温度 。 了不起。 但是...指针指向哪里？ 你不知道 temp的值（它包含的内存地址）是创建时恰好位于其在内存中的位置的垃圾（其他编程语言为未初始化的变量提供默认值，但C不在该组中）。

所以...你很幸运。 最好将您的程序描述为“ 未定义的行为 ”，因为temp可能包含一个随意占用的内存地址：您的程序可以通过该函数运行，在其他地方崩溃，并且您不知道原因。

另一个问题是，当函数返回void ，您应该返回新的结构用法。

这是应该的功能：

usage * addUse(int index,char *nm, int addr){
     usage *temp = malloc(sizeof(usage));

     if ( temp == NULL) {
         fprintf(stderr, "Not enough memory");
         exit(EXIT_FAILURE);
     }

     strcpy(temp->name,nm); //segfault here. 
     temp->loc = addr;
     temp->next= NULL;
    /* more processing */

     return temp;
}

我知道您应该将addUse（）返回的指针链接到链接列表。

希望这可以帮助。