Discussion :

[Fahmi06]

J'ai un petit bout de code qui me génère toujours une erreur de segmentation

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
 
if (neighbor->id == header->src) {
	      struct ID_REQUEST* id_req = (struct ID_REQUEST*)malloc(sizeof(ID_REQUEST));
	      id_req->Entier=nodedata->seq;
	      id_req->suivant = NULL;
	      if (neighbor->id_req == NULL){
		  neighbor->id_req = id_req;
		 printf ("\n hello1 \n"); 
	      }
	      else {
                  printf ("\n hello2 \n");
		  struct ID_REQUEST* temp=neighbor->id_req;
 
		  while(temp->suivant != NULL)
		  {
 
		      temp = temp->suivant;
		  }
		  temp->suivant = id_req;
 
	      }

Je n'arrive pas à voir le pb.
Merci d'avantage

[diogene]

Je ne vois rien d'anormal dans ce tronçon de code. Mais il reste à vérifier que neighbor, header et nodedata sont différents de NULL et correspondent à une adresse légale.

Un des messages "hello" est il affiché avant le plantage ?

Il serait très souhaitable de tester le retour du malloc.

[tonton fred]

Salut,
Probleme avec ton malloc()

Code :

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

1
2
3
4
5
6
7
8
9
/* allouer assez d'espace */
struct ID_REQUEST* id_req = malloc(sizeof(struct ID_REQUEST));
/* pour ne pas se tromper a l'avenir */
struct ID_REQUEST* id_req = malloc(sizeof(*id_req));
/* et TESTER le retour de malloc AVANT d'assigner les membres de la structure */
if (id_req != NULL ) {
	      id_req->Entier=nodedata->seq;
	      id_req->suivant = NULL;

[Médinoc]

Personnellement, j'utilise ceci:

Code C :

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

struct ID_REQUEST* id_req = malloc(sizeof *id_req);

[Fahmi06]

Merci pour vos réponses rapides
En fait j'ai tester les propositions de tonton fred et le même pb persiste.
Pour Médinoc : oui je trouve des hello 1 et de hello2 et parfois que des hello 1 et même aucune des deux (je n'arrive pas à trouver une explication).
Le bout de code devient :

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
 
if (neighbor->id == header->src) {
	      struct ID_REQUEST* id_req = malloc(sizeof(struct ID_REQUEST *));
	      if (id_req != NULL ) {
		  id_req->Entier=nodedata->seq;
		  id_req->suivant = NULL;
	      }
	      else { printf ("\n espace mémoire non suffisant \n");}
 
	      if (neighbor->id_req == NULL){
		  neighbor->id_req = id_req;
		 printf ("\n hello1 \n"); 
	      }
	      else {
		  printf ("\n hello2 \n");
		  struct ID_REQUEST* temp=neighbor->id_req;
 
		  while(temp->suivant != NULL)
		  {
 
		      temp = temp->suivant;
		  }
		  temp->suivant = id_req;
 
	      }

Pour être plus clair peu être. Ce bout de code est exécuté en distribué : je suis en fait en train de simuler un réseau de capteurs où chaque noeud veut ajouter dans une table de voisinage les sources des requêtes qu'il reçoive dans une liste chainée

[Trap D]

Il y a erreur ici :

Code :

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

struct ID_REQUEST* id_req = malloc(sizeof(struct ID_REQUEST *));

Tu lui demande d'allouer la taille d'un pointeur.
Tu devrais écrire

Code :

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

struct ID_REQUEST* id_req = malloc(sizeof(*id_req));

comme il me semble que celà t'a déjà été dit.

[Fahmi06]

J'ai changé cette ligne mais le pb persiste toujours

[diogene]

Code :

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

1
2
/* allouer assez d'espace */
struct ID_REQUEST* id_req = malloc(sizeof(struct ID_REQUEST));

Etonnant que le compilateur n'ai pas ralé si ID_REQUEST n'était pas défini. Si il était défini par autre chose que

Code :

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

typedef struct ID_REQUEST ID_REQUEST;

c'était suicidaire.

Code :

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

1
2
/* pour ne pas se tromper a l'avenir */
struct ID_REQUEST* id_req = malloc(sizeof(id_req));

Code :

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

struct ID_REQUEST* id_req = malloc(sizeof * id_req);