C 结构：分段错误

Question

关于结构的简单问题：

struct xint {
     int number;
     char string[12];
};

int main(int argc, char *argv[])
{
  struct xint offsets, *poffsets;
  poffsets=&offsets;
  FILE * pFile = fopen("file","rb");
  fread(poffsets,1,16,pFile);
  printf("Number %d\nString %s\n",offsets.number,offsets.string);
}

我得到这个输出

Number 12345
Segmentation fault

，我知道我可能在结构、指针和内存分配方面做错了一些事情。提前致谢 ：）

Answer 1

您的问题是您直接从文件中读入结构，而不检查结构对齐。改为这样做：

  fread(&offset.number,1,sizeof(offsets.number),pFile);
  fread(&offset.string,1,sizeof(offsets.string),pFile);

Answer 2

我怀疑您正在读取的文件数据没有以 NUL ('\0') 字符终止字符串。根据 C 标准库的 printf() 所遵循的字符串的 C 定义，字符串必须以 NUL 字符终止。

您可能会始终（通过代码）确保 .string[11] = '\0'。

或者，声明 string[13] 并确保 string[12] = '\0'

另外，另一位发帖者提到了结构成员对齐问题。这是您也必须解决的一个合理的担忧。

Answer 3

我猜测该字符串在文件中不是以空值终止的，并且您的代码也没有执行任何以空值终止的操作。

fread(poffsets, 1, 16, pFile);
offsets.string[11] = '\0';
printf("Number %d\nString %s\n", offsets.number, offsets.string);

或者修改文件，使字符串以空字节结尾。

Answer 4

你会得到缓冲区溢出。您的字符串包含 12 个字符，但没有空间用于终止 '\0'。

如果您这样做：

struct xint {
     int number;
     char string[16]; // Make sure you have enough space for the string + '\0'.
};

int main(int argc, char *argv[])
{
    struct xint offsets, *poffsets;

    // Initialize your memory to 0. This will ensure your string is 
    // '\0'-terminated.
    // FYI, sizeof(xint) here is 20.
    memset(&offsets, 0, sizeof(xint)); 

    poffsets=&offsets;
    FILE * pFile = fopen("file","rb");
    fread(poffsets,1,16,pFile);
    fclose(pFile);
    printf("Number %d\nString %s\n",offsets.number,offsets.string);
}

那就可以解决问题。