c程序中的分段错误,使用malloc(Linux os)

时间:2019-04-28 14:22:24

标签: c linux

此c程序可在Windows中运行,但在Linux中会出现“分段错误(核心转储)”。我猜错了错误指针或malloc函数。没有指针和malloc我无法返回结构体数组。

struct team {
    char name[12];
    int m_count;
    int score;
};

struct team *teamName(){        

    FILE *fp;
    fp=fopen("teams.txt", "r");

    struct team *items; 
    items= (struct team *) malloc(sizeof(struct team) * 10);

    int i;
    for(i=0; i<10; i++)
    {
        fscanf(fp, "%s\n" , items[i].name); 
    }

    fclose(fp);
    return items;
}

int main(void)
{   struct team *items = teamName();    
    getMatch(items);
}

1 个答案:

答案 0 :(得分:1)

您的代码中存在几个问题:

  • 您不检查打开成功

  • 您不会检查 fscanf 是否成功,并且如果读取名称大于11,则会以不确定的行为从缓冲区中写出

  • 为什么\n采用 fscanf 格式?

  • 如果您阅读的名称少于10个,则不会设置某些条目,以后可能会有不确定的行为


考虑到我的言论的提案可以是:

#include <stdio.h>
#include <stdlib.h>

struct team {
  char name[12];
  int m_count;
  int score;
};

struct team *teamName(){        
  FILE *fp = fopen("teams.txt", "r");

  if (fp == NULL)
    return NULL;

  struct team *items = malloc(sizeof(struct team) * 10);

  int i;

  for (i=0; i<10; i++)
  {
    if (fscanf(fp, "%11s" , items[i].name) != 1) {
      /* empty other names */
      do {
        items[i].name[0] = 0;
      }
      while (++i != 10);
      break;
    }
  }

  fclose(fp);
  return items;
}

int main(void)
{
  struct team *items = teamName();

  if (items != NULL) {
    for (int i = 0; i != 10; ++i) {
      if (items[i].name[0] != 0)
        puts(items[i].name);
    }
  }

  /* getMatch(items); */
}

编译和执行:

pi@raspberrypi:/tmp $ gcc -pedantic -Wextra -Wall m.c
pi@raspberrypi:/tmp $ cat teams.txt 
aze qsd
loop
bar
pi@raspberrypi:/tmp $ ./a.out
aze
qsd
loop
bar
pi@raspberrypi:/tmp $ 

请注意 fscanf 读过的单词,我的意思是名称不能包含空格,否则您需要使用实例 fgets