使用 fgetc() 时出现分段错误。我该如何解决？

Question

我想从文件流中读取 z 个字节。然后我想将 char 数组中的值返回。如果您能给我一个解释，我将不胜感激。从现在开始，这就是我的代码：

char * getData(FILE * fp, long * x)
{
  int z = 0;
  char * data = malloc(sizeof(char) * BUFFLENGTH);
  strcpy(data,"");

  while(z < BUFFLENGTH-2)
  {
    if(feof(fp) == 0)
    {
      data[z] = fgetc(fp);
      z++;
      x++;
    }
    else
    {
       strcat(data,"\0");
       return data;
    }
  }    
}

我知道触发了分段错误抛出了这一行：

data[z] = fgetc(fp);

但我不知道为什么。

Answer 1

• 您应该检查分配是否成功。
• 您应该在调用fgetc() 而不是使用feof() 之后检查读取是否成功。
• strcat() 的这种用法将调用 未定义的行为，如果从文件中读取一个或多个字节并且没有读取其值为 0 的字节，因为指针指向的不是以空值结尾的东西string 将被传递给strcat()，它将访问通过malloc() 分配的区域，而不是初始化。取而代之的是，您应该通过添加 '\0' 来终止字符串。
• 即使读取的长度超过限制，您也应该显式返回某些内容，否则如果调用者使用返回值，您将调用 未定义的行为。

应用了这些建议的代码：

char * getData(FILE * fp, long * x)
{
  int z = 0;
  char * data = malloc(sizeof(char) * BUFFLENGTH);
  if(data == NULL) return NULL;
  /* strcpy() isn't needed because it will be overwritten */

  while(z < BUFFLENGTH-2)
  {
    int input = fgetc(fp);
    if(input != EOF)
    {
      data[z] = input;
      z++;
      x++;
    }
    else
    {
       data[z] = '\0';
       return data;
    }
  }
  free(data);
  return NULL;
}