使用 strcat() 的分段错误

Question

在下面的代码中，我的 strcat 出现分段错误：

csv* lines_into_csv(int m,char* array[LINES_MAX][COLUMNS]){
   char *first_string, *final_string;
   first_string = (char*)calloc(400,sizeof(char));
   final_string = (char*)calloc(400,sizeof(char));
   csv *earthquake = malloc(sizeof(csv)*LINES_MAX);
   int n, j;

   for(j = 0; j < m; j++){
     for(n = 0; n < COLUMNS; n++ ){
       if (array[j][n] != NULL) {
         if(n < 4){
           strcat(first_string, array[j][n]);
        }
        else if(n == 4){

          earthquake[j].mag = atof(array[j][n]);
        }
        else {

          strcat(final_string, array[j][n]);
        }
       }
     }
     earthquake[j].start_of_line = (char*)calloc(200,sizeof(char));
     earthquake[j].end_of_line = (char*)calloc(200,sizeof(char));
     earthquake[j].start_of_line = strdup(first_string);
     earthquake[j].end_of_line = strdup(final_string);
     free(first_string);free(final_string);
   }


   return earthquake;
}

csv 类型结构由两个 char* 和一个浮点数组成，浮点数是我对文件进行排序的数字。

Answer 1

这里有代码

csv *earthquake = malloc(sizeof(csv*)*LINES_MAX);

您被分配了一个指向 csv 结构的指针数组，但您似乎假设您可以在此处访问 struct csv 的对象：

earthquake->start_of_line = (char*)calloc(200,sizeof(char));
earthquake->end_of_line = (char*)calloc(200,sizeof(char));

你可能是说

csv *earthquake = malloc(sizeof(csv)*LINES_MAX); 

编辑：

检查参数中的长度不超过您分配的长度。还要确保 in-strings 正确地以 0 结尾。

Answer 2

您对strcat() 的使用存在问题，因为您没有初始化任何用作目标的字符串。

strcat(first_string, tmp);
...
strcat(final_string, tmp);

strcat() 期望找到一个 nul 终止符来追加下一个字符串。如果它没有找到一个，它会一直在内存中翻找直到找到，那么谁知道它将把你的另一个字符串复制到哪里呢？

一种解决方案是使用calloc() 将0 写入它分配的内存。

还有另一个问题，内存泄漏。您已在此处分配内存 - 但仅用于第一个数组元素。

earthquake->start_of_line = malloc( 100*sizeof(char*));
earthquake->end_of_line = malloc( 200*sizeof(char*));

这会在你做完之后很快导致内存泄漏

earthquake[j].start_of_line = strdup(first_string);
earthquake[j].end_of_line = strdup(final_string);

您应该知道strdup() 分配了更多内存并返回其指针，然后您会覆盖malloc() 返回的原始指针，因此无法释放该内存，因为您不再拥有指针。

转到您用于dup() 的函数参数array[][]，它是初始化的还是随机的？