Discussion :

[adrienmgs]

Bonjour, en général j'arrive à régler seul les erreurs de segmentation qui découlent souvent d'erreurs d'inattention mais là je dois avouer que je n'y comprend rien, mais alors absolument rien...

Je vous met le code concerné :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
//Définition de ma structure :
typedef struct trajet *Trajet;
struct trajet
{
	int ligne;
	int v_a;
	int h_d;
	int h_a;
	Trajet suivant;
};
 
//Declaré en variable globale :
Trajet *Liste;
 
//Dans la fonction qui plante
if(Liste[0]==NULL) printf("OK\n");
h_d=lire_horaire();
if(Liste[0]==NULL) printf("OK\n");
 
//Et la fonction lire_horaire() :
int lire_horaire()
{
	int h_t;
	int heures;
	int minutes;
	char trash='h';
	fscanf(fichier, "%2d", &heures);
	fscanf(fichier, "%1s", &trash);
	fscanf(fichier, "%2d", &minutes);
	h_t=60*heures+minutes;
	return h_t;
}

Ici le premier test sur Liste[0] permet bien d'afficher "OK", mais le deuxième provoque une segmentation fault...
Or si je peux le tester avant de lancer ma fonction lire_horaire() qui n'a rien à voir avec ce tableau Liste, pourquoi ne le puis-je plus après ?

Si quelqu'un à une idée je suis preneur, je commence à désespérer...

Merci d'avance !

Adrien.

[jeroman]

Salut

Je peux me tromper, mais ça sent le débordement de tampon. Des bugs sans queue ni tête de cette nature ont souvent cette origine. Toutefois, souvent ne veut pas dire toujours...
Un débordement (en fonction de là où il se situe) peut modifier une autre variable, ou même éventuellement le registre EBP empilé sur la pile. Et une fois que la fonction se termine... bing !

Y a déjà un beau débordement ici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
char trash='h';
...
fscanf(fichier, "%1s", &trash);

La fonction rajoute un '\0' après le caractère. Vu que la variable ne fait qu'un byte, ça déborde d'un byte... et ça écrase ce qui s'y trouve (une autre variable, un registre de pile précédemment empilé... )

Formateur "%s" :
* s'arrête au premier caractère "blanc" (sauf si l'on a précisé une longueur de champ, et que celle-ci est atteinte avant un blanc.)
* supprime les espaces lors de l'analyse
* ajoute un terminateur de chaîne (caractère '\0') de façon automatique.

Je ne dis pas que ton problème vient forcément de là, mais c'est déjà un débordement, avec toutes les conséquences que ça peut occasionner.

Par contre, rien à voir, mais ça sent les globales à plein nez. Les globales sont la plupart du temps injustifiées et sont susceptibles elles aussi de rendre le code indebuggable (n'importe quelle fonction peut changer n'importe quoi).

[adrienmgs]

C'est surement ça j'ai voulu gagner du temps en déclarant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char trash='h';

je pensais que ça prendrait automatiquement en compte le \0 en créant un tableau de 2 char, mais après reflexion ....

Pour ce qui est des variables globales j'ai d'abord essayé avec 100% des variables en local puis j'en ai passé quelques unes en global pour voir si ça influait sur mon problème mais c'était sans effet je vais donc tout pouvoir repasser en local.

Merci de ta réponse rapide, je vais de ce pas vérifier si ça résoud mon problème !

Edit : en effet ! bien vu et encore merci,

Si jamais la modif que j'ai fait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
//dans lire_horaire()
char *trash=NULL;
trash=malloc(2*sizeof(char));

[diogene]

Pourquoi faire de l'allocation dynamique ? Il suffit de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char trash[]="h";

ou même, puisque trash n'est pas utilisé ensuite, ne pas stocker ce champ :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
	fscanf(fichier, "%2d", &heures);
	fscanf(fichier, "%*1s");
	fscanf(fichier, "%2d", &minutes);