2016-12-02 53 views
-1

首先让我为格式化和难度代码道歉。我是C和Stack的新手。这里大部分的混乱代码可能与问题无关,但必须包含在上下文中。realloc期间的分段错误

在第一次调用realloc(注释中注明)后,下面的代码会出现分段错误。 return_file->target_line只是一个3D数组,i是3D数组第一维的元素数。所以我打电话realloc来存储额外的二维数组(类型char **)。

NULL内存分配的返回被故意省略B/C开发协议明确指出所有的内存分配将会成功(我有疑问)。

我正在使用自己的内存检查程序。错误代码我得到的是:

Program received signal SIGSEGV, Segmentation fault. 
0x00007ffff7ac63fb in reallochook() from /lib64/libc.so.6 

我已经看了它很长一段时间,但似乎无法找出问题所在。

Mockfile *read_mockfile(const char filename[]) { 
    Mockfile *return_file = NULL; 
    FILE *input; 

    if(filename != NULL && (input = fopen(filename, "r")) != NULL) { 
    char **split_tmp, line[MAX] = {0}; 

    return_file = malloc(sizeof(Mockfile)); 
    return_file->rule_count = 0; 

    /*read lines*/ 
    while(fgets(line, MAX, input) != NULL){ 
     if(line[0] != '#' && line[0] != '\n'){ 
     int j, i = return_file->rule_count; 
     split_tmp = split(line); 

     if(line[0] != '\t'){ 
      j = 0; 


      /*target line. Realloc every string in three steps. Segementation fault occurs after this line below.*/ 
      return_file->target_line = realloc(return_file->target_line, (i + 1) * sizeof(char **)); 

      while(split_tmp[j] != NULL){ 
      return_file->target_line[i] = realloc(return_file->target_line[i], (j + 1) * sizeof(char *)); 
      return_file->target_line[i][j] = malloc(strlen(split_tmp[j]) + 1); 
      strcpy(return_file->target_line[i][j], split_tmp[j]); 
      j++; 
      } 
      return_file->target_line[i] = realloc(return_file->target_line[i], (j + 1) * sizeof(char *)); 
      return_file->target_line[i][j] = NULL; 
     } else { 
      j = 0; 

      /*action line. Allocate every string in three steps*/ 
      return_file->action_line = realloc(return_file->action_line, (i + 1) * sizeof(char **)); 

      while(split_tmp[j] != NULL){ 
      return_file->action_line[i] = realloc(return_file->action_line[i], (j + 1) * sizeof(char *)); 
      return_file->action_line[i][j] = malloc(strlen(split_tmp[j]) + 1); 
      strcpy(return_file->action_line[i][j], split_tmp[j]); 
      j++; 
      } 
      return_file->action_line[i] = realloc(return_file->action_line[i], (j + 1) * sizeof(char *)); 
      return_file->action_line[i][j] = NULL; 

      return_file->rule_count++; 
     } 
     } 
    } 
    fclose(input); 
    } 
    return return_file; 
} 
+0

@MD XF,你刚刚在谈论大括号不在新行吗? B/C这是我故意保留的个人偏好。 –

+0

无论为你工作......适合你。但是还有一些关键的改进,一些小的细节造成了很大的差异 –

回答

5

realloc()预计其第一个参数指向的内存或NULL有效的块,所以malloc()后,你应该初始化:

return_file = malloc(sizeof(Mockfile)); 
return_file->rule_count = 0; 
return_file->target_line = NULL; /* Add this */ 

这应该解决崩溃。

还请注意,foo = realloc(foo, N);是一个错误,因为realloc()可以返回NULL,所以你需要处理完整性。

+0

背后的原因是什么?是否需要其他分配功能?即'malloc()'和'calloc()'? –

+1

你指的是哪个“这个”? C标准定义了行为[如本页所述](http://en.cppreference.com/w/c/memory/realloc);对'malloc()'来说'NULL'返回值检查当然是必要的,但是出于不同的原因。 –