Seg Fault sur un fclose

Version imprimable

30/03/2010, 19h24
XstasY

Seg Fault sur un fclose

Bonjour,

Je vous écris car j'ai un petit soucis dans une fonction d'un projet d'etude.
J'ai cree une fonction permettant de recuperer des données texte dans un fichier, mais lors de l'execution, celui ci me renvoie un seg fault ...
Apres utilisation de gdb, je me rends compte que le signal est envoye par la fonction fclose, en enlevant celle ci, tout fonctionne parfaitement et mes données sont parfaitement stockées la ou je le souhaite.

Auriez vous une idée d'ou cela peut il venir?

Je vous remercie pour votre aide :)
30/03/2010, 19h35
supersnail

Bonjour et bienvenue sur le forum,

Pourrait-on voir ton code? (entre les balise [code] bien entendu ;) )

le voila :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
t_graph lecture_fichier ()
{
  FILE *fichier = NULL;
  char line[BUFSIZ];
  int s,d;
  float c;
  t_graph g = NULL;
  if (NULL ==(fichier = fopen("tmp.dj", "r")))
  {
    fprintf(stderr,"File not found\n");
    exit(EXIT_FAILURE);
  }
  while (fscanf(fichier,"%s %d %f %d",line,&s,&c,&d) != EOF)
  {
    //Traitement sans toucher au fichier
  }
  if (!feof(fichier))
  {
    fprintf (stderr, "Probleme durant la lecture\n");
    exit(EXIT_FAILURE);
  }
  fclose(fichier);  // voici l'endroit ou le seg fault est signale par gdb
  return(g);
}

30/03/2010, 21h02
Obsidian

Il est probable que les données que tu lis, et plus précisément la chaîne en début de scanf(), soit plus longue que BUFSIZ, déborde du tampon alloué, et aille écraser le pointeur « fichier » déclaré juste à côté. Du coup, lorsque tu veux l'utiliser dans ton close(), il pointe une adresse complètement farfelue.

Retirer le close() empêche donc l'erreur de se produire, puisque c'est le système qui va faire le ménage à ta place, mais ne résout pas le problème pour autant : les variables proches sont toujours écrasées.
30/03/2010, 21h09
ram-0000

je pense que pour lire un float, il faut un %lf et pas un %f
30/03/2010, 21h18
gl

Citation:

Envoyé par ram-0000

je pense que pour lire un float, il faut un %lf et pas un %f

Non, c'est bien %f, %lf c'est pour un double.
30/03/2010, 21h36
XstasY

Citation:

Il est probable que les données que tu lis, et plus précisément la chaîne en début de scanf(), soit plus longue que BUFSIZ

Je ne pense pas, car la taille de la macros BUFSIZ est de environ 8000 et mes chaines sont ( au maximum ) de 11 caracteres ( 12 avec les /n ) et j'ai au maximum, pour l'instant, une vingtaine de lignes a lire ( l'utilisation de BUFSIZ vient du fait que par la suite, les fichiers peuvent atteindre les 1000 lignes )
30/03/2010, 22h48
gl

Citation:

Envoyé par XstasY

Je ne pense pas, car la taille de la macros BUFSIZ est de environ 8000 et mes chaines sont ( au maximum ) de 11 caracteres ( 12 avec les /n ) et j'ai au maximum, pour l'instant, une vingtaine de lignes a lire ( l'utilisation de BUFSIZ vient du fait que par la suite, les fichiers peuvent atteindre les 1000 lignes )

Dans le code présenté, je ne vois pas de raisons, autre que celle suggérée par Obsidian, d'avoir ce problème.

Peux-tu fournir un code minimal mais compilable reproduisant ce problème ainsi qu'un fichier entrée d'exemple ?
31/03/2010, 00h11
Obsidian
J'ajoute également que tu peux contrôler la taille de BUFSIZ pour voir si elle est réellement définie comme tu le crois. D'autre part, si tu ouvres le mauvais fichier par exemple, la lecture de la chaîne se poursuivra tant que la fonction ne rencontrera ni un espace, ni un terminateur de chaîne.

Essaie d'ajouter les lignes suivantes tout en haut de ton bloc while (scanf(…)) :
Code:

1 2 printf ("Taille de la ligne : %d sur %d.\n",strlen(line),BUFSIZ); assert (strlen(line)<BUFSIZ);