Нарушение сегментации: while (fgets (buffer, 15, statfile)! = NULL) {};

Question

Нарушение сегментации: while (fgets (buffer, 15, statfile)! = NULL) {};

0

Этот код работает для нескольких тысяч вызовов в строке, если r адресуется последовательно. Он работает несколько сотен подряд, если r выбран случайным образом.

Однако в какой-то момент случайных r я получаю нарушение сегментации. Я не уверен, что может быть причиной этого, поскольку он отлично работает почти все время.

*** Break *** segmentation violation
Segmentation fault

Авария происходит в fgets.

int fsize(int r, double VME, std::string* filepath) {
  FILE *statfile;
  std::stringstream streamfilename;
  std::stringstream streamcommand;
  string* filename = new string;
  std::string command;
  char buffer [15];
  char charsize;
  streamfilename.str("");
  streamfilename << *filepath << "run" << r << "-" << VME << ".dat";
  *filename = streamfilename.str();
  streamcommand.str("");
  streamcommand << "stat -c " << "%s " << *filename;
  command = streamcommand.str();
  const char * charcommand = command.c_str();
  statfile = popen(charcommand, "r");
  cout << "Test 5\n";
  while(fgets(buffer,15,statfile)!=NULL){}; CRASH OCCURS HERE
  cout << "Test 6\n";
  int* filesize = new int;
  *filesize = atoi(buffer);
  pclose(statfile);
  return *filesize;
}

В чем причина ошибки сегментации?

Machinus 04 янв. 2014, в 20:05

Источник

0

Действует ли statfile? Более того ... зачем смешивать C и C ++?
Adriano Repetti 04 янв. 2014, в 18:48
0

Если вы проверите возвращаемое значение popen (), вы, скорее всего, увидите, что оно не удалось. statfile будет NULL, и это ваш путь к краху.
TimDave 04 янв. 2014, в 18:49
0

Адриано: Можете ли вы быть более конкретным о Си? Я не уверен, что вы имеете в виду.
Machinus 04 янв. 2014, в 19:03
0

I am not sure what could be causing this since it works fine almost all of the time. «Работать почти все время» не является показателем чего-либо. Последствия повреждения памяти непредсказуемы.
Lightness Races in Orbit 04 янв. 2014, в 19:04
0

Что бы вы предложили в качестве замены для fgets и popen? Я не против переписать метод, но я не уверен, что лучше.
Machinus 04 янв. 2014, в 19:07
0

Вы должны использовать отладчик и посмотреть, какие переменные имеют неожиданное значение. Вероятно, popen подходит для того, что вы пытаетесь сделать, но это может не сработать.
zneak 04 янв. 2014, в 19:09
0

Переменная «размер файла» предполагает, что этот код был написан с недостаточным пониманием указателей и управления памятью. Заставляет меня бояться худшего для остальной части программы. Время для серьезных обзоров кода.
Ruud Helderman 04 янв. 2014, в 19:16
0

Радд: В чем проблема с размером файла?
Machinus 04 янв. 2014, в 19:17
1

Вы пропускаете filesize . filename тоже. В вашем коде есть два появления new (без уважительной причины), но ни одно из них не было delete .
Igor Tandetnik 04 янв. 2014, в 19:51
1

Игорь прав. Эти 3 строки кода: int* filesize = new int; *filesize = atoi(buffer); return *filesize; можно заменить только одной строкой: return atoi(buffer); , Если это над вашей головой, вам нужно время, чтобы учиться. Я никого не осуждаю, этот материал трудно понять, но он необходим при программировании на C / C ++.
Ruud Helderman 04 янв. 2014, в 23:19

Показать ещё 8 комментариев

Теги:

c++

segmentation-fault

fgets

1 ответ

Ещё вопросы

Действует ли statfile? Более того ... зачем смешивать C и C ++?
Если вы проверите возвращаемое значение popen (), вы, скорее всего, увидите, что оно не удалось. statfile будет NULL, и это ваш путь к краху.
Адриано: Можете ли вы быть более конкретным о Си? Я не уверен, что вы имеете в виду.
I am not sure what could be causing this since it works fine almost all of the time. «Работать почти все время» не является показателем чего-либо. Последствия повреждения памяти непредсказуемы.
Что бы вы предложили в качестве замены для fgets и popen? Я не против переписать метод, но я не уверен, что лучше.
Вы должны использовать отладчик и посмотреть, какие переменные имеют неожиданное значение. Вероятно, popen подходит для того, что вы пытаетесь сделать, но это может не сработать.
Переменная «размер файла» предполагает, что этот код был написан с недостаточным пониманием указателей и управления памятью. Заставляет меня бояться худшего для остальной части программы. Время для серьезных обзоров кода.
Радд: В чем проблема с размером файла?
Вы пропускаете filesize . filename тоже. В вашем коде есть два появления new (без уважительной причины), но ни одно из них не было delete .
Игорь прав. Эти 3 строки кода: int* filesize = new int; *filesize = atoi(buffer); return *filesize; можно заменить только одной строкой: return atoi(buffer); , Если это над вашей головой, вам нужно время, чтобы учиться. Я никого не осуждаю, этот материал трудно понять, но он необходим при программировании на C / C ++.

mpromonet · Accepted Answer · 2014-01-04T17-16-00.000Z

statFile, вероятно, NULL.

Если вы хотите получить размер файла, вы должны рассмотреть функцию C:

 int stat(const char *path, struct stat *buf);

Это позволит избежать разветвления команды и анализа ее результата.