Discussion :

[LCoileux]

Bonjour à tous,
voilà je me creuse depuis un moment deja sur une erreur de segmentation que je ne comprend pas.

En gros, j'ai un fichier que je parse, je recupere les données dans des variables, et je veux les stocker dans une structure. Ma structure est celle-là :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
struct process{
 
	int pid;
	int pidPere;
	char *user;
	char *nomCommande;
};
 
typedef struct process PROCESS;

Et mon code :

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
 
GNode *creeArbre(char *Chaine){
 
	PROCESS *pProc;
	pProc = (PROCESS *)malloc(sizeof(PROCESS));
	GNode *noeud;
	GNode *racine;
 
        FILE *fp = NULL;
 
	if((fp=fopen(Chaine,"r"))==0){
   		menu(); /*retour menu*/
	}
 
	fgets(ligne,LONG,fp);/*up la premiere ligne*/
 
	while (fgets(ligne,LONG,fp) != 0){
		sscanf(ligne,"%s %d %d %d %s %s %s %s", pUser, &pid, &ppid, &c, stime, tty, time, pNomCommande);
		pProc->user = pUser;	 /*C'est là que ca merde*/
		pProc->pid = pid;
		pProc->pidPere = ppid;
		pProc->nomCommande = pNomCommande; 
		g_node_insert( g_node_find_pid(racine,pProc->pidPere),-1,g_node_new(pProc));*/
	}

Mon sscanf me recupere bien toutes les données, mais l'affectation dans le champ de la structure loupe. Le plus étrange, c'est que si je fais l'affectation avant la boucle while, elle fonctionne, mais si je l'a fait dedans ou apres, non.

Alors je ne sais pas si ca peut venir du sscanf, parce que quand j'affiche ce que j'ai recuperé, c'est bon...

[D[r]eadLock]

Alors je ne sais pas si ca peut venir du sscanf, parce que quand j'affiche ce que j'ai recuperé, c'est bon...

Ce n'est pas parce que ce qui est affiché est correct, que le code l'est ;D

Il n'y a pas la déclaration des pointeurs donné à sscanf (pUser, ...). Ces pointeurs sont-ils alloués ? À quelle taille ? Car ce qui est douteux est qu'il n'y a pas de limite sur les %s. Tu peux utiliser valgrind pour voir si tu ne fait pas des trucs "louches".
En plus, dans ta boucle tu va faire pointer user et nomCommande sur une même adresse.

[LCoileux]

C'est vrai que je n'ai pas mis mes declarations, j'ai jugé inutile de les poster car (a mon avis) ca ne vient pas de là...
En tout cas, les voilà :

Code :

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

1
2
3
4
5
 
#define LONG 1000
 
char stime[LONG],tty[LONG],time[LONG], pUser[LONG], pNomCommande[LONG], ligne[LONG];
int pid,ppid,c;

Comme debuggueur j'ai utilisé ddd, celui-ci me disait que l'erreur venait de sscanf mais je ne vois pas pourquoi.

[jeroman]

Salut

Ton code est blindé de globales. Elles ne sont absolument pas justifiées... et ça rend le debuggage encore plus compliqué, vu qu'on ne peut plus vraiment savoir qui modifie quoi.

Y a un truc qui me surprend dans ton code :

Code :

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

1
2
3
4
5
6
if((fp=fopen(Chaine,"r"))==0){
	menu(); /*retour menu*/
}
 
fgets(ligne,LONG,fp);/*up la premiere ligne*/
(...)

Lorsque tu appelles la fonction menu en cas d'échec de fopen, tu provoques bien un appel... pas un saut. Lorsque la fonction menu se termine, l'exécution du code reprend juste après l'appel de cette fonction, c'est-à-dire en faisant crasher fgets, vu que fp est égal à NULL...

[LCoileux]

Ah merde, j'ai oublié le else, quel ***
Mais en fait, ma fonction menu attend une entrée de ma part, du coup c'est pas sur que je revienne dans cette partie du code. A moins qu'elle y reviendra quand meme ?

Pour les globales, je n'etais pas vraiment sur de pouvoir remplir directement les champs de ma structure avec sscanf, donc dans le doute j'ai preféré passer par ca.

[diogene]

De toute façon, ceci va provoquer des trucs baroques :

Code :

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

1
2
3
  pProc->user = pUser;	 /*C'est là que ca merde*/
....
  pProc->nomCommande = pNomCommande;

Il y a recopie de l'adresse des tableaux pUser et pNomCommande, pas recopie de leur contenu (d'ailleurs, il n'y a pas de place dans la structure pour stocker une copie).
Comme on utilise toujours les deux mêmes tableaux, toutes les structures stockeront les mêmes adresses et pointeront donc vers les deux mêmes chaines, à savoir les dernières lues.

[D[r]eadLock]

Bon, reprenons (avec les remarques de jeroman et diogène), pour essayer d'y voir plus clair:

• la valeur de retour du malloc n'est pas vérifiée
• il y trop globales et c'est mal(tm)
• pProc n'est alloué qu'une seule fois, alors qu'il semblerait qu'il faille insérer plusieurs entrées dans l'arbre
• racine n'est pas déclaré (y'a pas de warnings la dessus ?, ou code manquant ?)
• le code du premier fgets n'est pas testé
• le code de retour du sscanf n'est pas testé
• les données parsées ne sont pas dupliquées (pProc->nomCommande = strdup(...))
• les codes de retour de g_node_{insert,find_pid,new} ne sont pas testés
• le protoype devrait sûrement être GNode *creeArbre(const char *Chaine)
• si le test de l'ouverture échoue, on continue au lieu de s'arrêter (un return NULL serait pas mal)
• fgets renvoie un pointeur, ce serait bien de le tester avec NULL (et non 0)
• time et stime sont des fonctions de la libc, ce serait bien d'éviter d'utiliser leur nom
• code manquant ?:
  • noeud n'est pas utilisé
  • il y a un */ qui traîne
  • pas de fclose(fp)

Maintenant, une fois que ceci sera corrigé , quelques informations complémentaires permettraient d'avancer si le problème est toujours présent:
- cela se produit-il au premier passage de boucle ?
- sinon, quand ? Sur une ligne particulière ?
- je persiste à penser que valgrind pourrait t'aider si tu fais de mauvaise choses en mémoire; les effets ne sont pas visible tout de suite et le programme peut ne pas planter ou planter à un endroit tout à fait différent de là où est l'erreur.