Почему появляется ошибка segmentation fault но при повторном запуске программы её нет?

Question

[vladislav9619996]

Мне нужно включать и выключать gpio38 для этого использую код на с++ но раз через раз выскакивает ошибка segmentation fault (то есть первый раз запустил все нормально второй раз запустил ошибка третий все нормально четвёртый ошибка и так далее причем в коде ничего не меняя)

/* gpio38.c */
#include <stdio.h>
#include <stdlib.h>
#define GPIO_PATH   "/sys/class/gpio"
#define GPIO38_PATH “/sys/class/gpio/gpio38”
 
int main( int argc, char * argv[] )
{
   FILE * fp;
 
  int a=0;
 
  fp = fopen( GPIO_PATH"/export","w");
  
  if ( fp == NULL ) {
      printf(“can not open file\n”);
      return EXIT_FAILURE;
  }
  fprintf( fp, "38");
  fclose( fp );
 
  fp = fopen( GPIO38_PATH”/direction”, “w”);
 
  if ( fp == NULL ) {
      printf(“can not open file\n”);
      return EXIT_FAILURE;
  }
 if(a==1)
  fprintf( fp, “high”);
else fprintf( fp, “low”);
  fclose(fp);
 
  fp = fopen( GPIO38_PATH”/value”, “w”);
  
  if ( fp == NULL ) {
      printf(“can not open file\n”);
      return EXIT_FAILURE;
  }
 
  fclose(fp);
}

Подскажите в чем ошибка не могу понять почему такое странное поведение

Comments

[jcmvbkbc]

перед последней } добавьте return 0; (к ошибке не относится, фен-шуй)

hint000, выход из int main через закрывающую скобку -- вполне по фен-шую по стандарту.

Answer 1

[jcmvbkbc]

Я не вижу криминала который мог бы привести к segfault в приведённой программе.
Почему поведение может отличаться между запусками -- например потому что запись в файл /sys/class/gpio/export меняет состояние системы и запоминается между запусками.
Кроме того, согласно https://www.kernel.org/doc/Documentation/gpio/sysfs.txt в direction надо писать in или out, а high или low нужно писать в value. Сегфолта от этого быть не должно, тем не менее.

Что происходит, если собрать программу с отладочной информацией и запустить так, чтобы она упала под отладчиком?

UPD: посмотрел ещё немного... В приведённом исходном тексте кривые юникодные кавычки. Рекомендую исправить на ASCII, либо вставить исходный текст как есть, без фокусов.

Comments

[vladislav9619996]

Он вылетает сразу после того как идёт проверка открыт ли файл direction выскакивает ошибка не может открыть и файл закрывается

[vladislav9619996]

В отладчике вовремя вознкновения ошибки в строке fp = fopen( GPIO38_PATH”/direction”, “w”);
fp равен 0 из за этого все валится на другой раз результат нормальный

[jcmvbkbc]

в строке fp = fopen( GPIO38_PATH”/direction”, “w”); fp равен 0 из за этого все валится

vladislav9619996, чё-то я не понимаю, у тебя там стоит проверка что fp == NULL и выход если так. Что валится-то? Завершает программу и выходит с кодом 1 -- это я понимаю. Но ты же говоришь segfault -- покажи, где именно.

Если проблема в том, что файл GPIO38_PATH”/direction” появляется не сразу после записи в GPIO_PATH"/export" -- ну возьми этот fopen в цикл, пока не откроется или пока не пройдёт время, которое ты готов ждать.

Answer 2

[nApoBo3]

Попробуйте использовать другой указатель для второго файла или добавить sleep между закрытием и открытием.

Comments

[CHolfield]

sleep надо применять там, где точно известно время ожидания. Данном случае вместо засыпания нужно ждать освобождения хендла файла, а не надеяться, что это случится.

[nApoBo3]

CHolfield, там по идее освобождения файла ждать не надо, там разные пути.
Но есть стойкое ощущение, что ошибка связанна именно с этим и скорее всего при параллельном выполнении.
Для того, чтобы понять в чем действительно дело нужны достаточно глубокие знания того как именно это работает на уровне ОС которыми я не обладаю.
Добавление sleep или использование иного указателя позволяет быстро проверить в этом ли дело. Понятно, что sleep после проверки нужно будет убрать.

[CHolfield]

nApoBo3, моя мысль была не настолько глубока. Я всего лишь хотел сказать, что если хочешь дождаться освобождения ресурса, то жди освобождения ресурса, а не 100мс - это принципально. Исходник не изучал)

Answer 3

[Andrej Gessel]

Почему программа падает с SIGSEGV мне тоже не совсем понятно.

В отладчике вовремя вознкновения ошибки в строке fp = fopen( GPIO38_PATH”/direction”, “w”);
fp равен 0 из за этого все валится на другой раз результат нормальный

Между экспортом и использованием (fopen(...direction)) стоит использовать небольшую паузу, ядру нужно немного время, что-бы экспортировать gpio.

Ну и странные кавычки конечно как сказал jcmvbkbc. может стоит компилировать так:
gcc main.c -Wall -Wpedantic

Так же можно проверить закрываются ли файлы, но это всё-равно не даёт SIGSEGV.

Answer 4

[shmelevka]

Эм могу ошибаться, но разве не надо почистить ь fp до того как заново использовать. Таким образом вы назначаете fp указателем на новый файл, а в области памяти остались остатки от прошлого. delete в помощь.