probleme de segfault

Version imprimable

bonjour a tous

alors voila mon programe avec des manipulation de fichier dont j'ai comme erreur : erreur de segmentation

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
FILE *tab=NULL;
FILE *rep=NULL;
FILE *file1,*file2,*file3;
char *p = NULL ;
char lecture[100],lit[100];
char *param;
char racine[100];
char fp[100],fp2[100];
 
 
 
 param="/opt/local/proddm/newcat/niveaufin.tab";
tab=fopen(param,"r"); 
fgets(lecture,100,tab);
strcpy ( racine, "/opt/local/proddm/newcat/");
if ( (p = strchr(lecture, '\n')) != NULL )
       *p = '\0' ;
 
 
 strcpy(fp,racine); 
 strcat(fp,lecture);
 
 file1=fopen(fp,"r");     
 strcat(fp,"/cst");
 
 file2=fopen(fp,"r");
 strcpy(fp2,fp);
 
 strcat(fp,"/dxf.cst");
 file3=fopen(fp,"r");    
 
while (!feof(tab))
  {
   if (file1==NULL || file2==NULL) 
      sprintf(lecture,"fond");	
   if (file3!=NULL) 
   {
 
   rep=fopen(fp,"r");
   } 
   else
   {
 
   strcat(fp2,"/couleur.cst");
   rep=fopen(fp2,"r");
   }
fgets(lit,100,rep);
 
}
fclose(file1);
fclose(file2);
fclose(file3);
fclose(rep);
fclose(tab);
}

si quelqu'un a une idee de l'origine du probleme

merci

11/05/2007, 09h20
Médinoc

C'est très confus, mais ce que je vois pour commencer, c'est que tu ne testes pas le retour de fopen(), et que tu n'emploies pas feof() correctement.

De plus, param devrait être un const char *.

PS: Et tes strcat() semblent générer au moins un chemin où le / intermédiaire est doublé.
11/05/2007, 09h30
acik59

pour ce qui est du / intermediaire j'ai verifier et je ne l'est jamais double

pour ce qui est du fopen je les tester c bon

maintenant comment utiliser correctement la fonction feof()
11/05/2007, 09h45
crocodilex

Appeler fopen() plusieurs fois dans une boucle while() n'est pas très malin....;)
11/05/2007, 09h48
crocodilex

Citation:

Envoyé par acik59

maintenant comment utiliser correctement la fonction feof()

Il me semble qu'on en avait déjà parler dans un de tes posts : http://www.developpez.net/forums/sho...d.php?t=333757

Une autre réponse dans la FAQ : ici

Citation:
Envoyé par acik59
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 strcpy(fp,racine); strcat(fp,lecture); file1=fopen(fp,"r"); strcat(fp,"/cst"); file2=fopen(fp,"r"); strcpy(fp2,fp); strcat(fp,"/dxf.cst"); file3=fopen(fp,"r");
Mis a part tout ce qui a deja ete signale (particulierement le test manaquant sur les ouvertures de fichiers), il y a un truc qui me chiffone dans ce que je vois au dessus, qu'essaies tu de faire exactement ?
La j'ai l'impression (de la maniere dont sont construit les noms de fichier) que tu cherches a ouvrir des repertoires.

11/05/2007, 10h41
acik59

pour le file 1 c'est en effet un test d'ouverture de repertoire!
11/05/2007, 10h47
Fabllot

Ce serait un peu plus joli...

http://c.developpez.com/faq/c/?page=...IERS_dir_exist
11/05/2007, 11h25
gl

Citation:

Envoyé par acik59

pour le file 1 c'est en effet un test d'ouverture de repertoire!

file2 aussi non ?

Sinon je doute fortement que fopen() sur un repertoire fonctionne bien. Et en plus c'est inutile, si un des repertoires n'existe pas, le fichier n'existera pas non plus (ben oui, ou le mettre) et son ouverture echouera.

En outre, si ne ouverture a echouee, le pointeur vaut NULL et fclose(NULL) est un comportement indetermine (c'est probablement la cause de ton segfault)
11/05/2007, 23h29
Didier77

pour les erreur de segmentation utilise gdb c'est mieux