Segmentation fault incomprehensible

**sorry60** · 10/12/2005, 16h44

Bonjour,

J'ai un segmentation fault assez bizarre, et surtout tres ennuyeux car je n'arrive pas à le résoudre

J'utilisa la structure suivante :

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
	typedef struct
	{
		GtkWidget *pFenetre;			/* Fenetre principale */
		GtkWidget *pLbIp;					/* Label IP */
		GtkWidget *pLbPort;			/* Label Port */
		GtkWidget *pLbLogin;			/* Label Login */
		GtkWidget *pLbMdp;			/* Label Mot de passe */
		GtkWidget *pLbDest;			/* Label destinataire */
		GtkWidget *pLbMsg;			/* Label message */
		GtkWidget *pZsIp;				/* Zone saisie IP */
		GtkWidget *pZsPort;			/* Zone saisie Port */
		GtkWidget *pZsLogin;			/* Zone saisie Login */
		GtkWidget *pZsMdp;			/* Zone saisie Mot de passe */
		GtkWidget *pZsDest;			/* Zone saisie destinataire */
		GtkWidget *pZsMsg;			/* Zone saisie message */
		GtkWidget *pBtConnect;	/* Bouton de connection */
		GtkWidget *pBtQuit;				/* Bouton quitter */
		GtkWidget *pBtSend;			/* Bouton d'envoie */
		GtkWidget *pZtInfos;			/* Zone de texte */
		SOCKET maSocket;
	}data;

Voici la fonction qui pose probleme :

Code :

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

1
2
3
4
5
6
7
void connection(gpointer donnees)
{
	data* pData = donnees;
 
	/*Creation socket */
	pData->maSocket = socket(PF_INET,SOCK_STREAM,0);
}

gpointer = void *
Le main :

Code :

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

1
2
3
4
5
6
7
8
int main(int argc, char* argv[])
{
   /* Declaration des variables */
   data pData;
   /*Connection du signal click de chaque bouton à la fonction appropriée*/ 	g_signal_connect(G_OBJECT(pData.pBtConnect),"clicked",G_CALLBACK(connection),&pData);
   /* ... */
   return 0;
}

Je ne comprend pas du tout pourquoi j'ai un segmentation fault lors de l'instruction :

Code :

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

pData->maSocket = socket(PF_INET,SOCK_STREAM,0);

Surtout que quand je fais :

Code :

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

pData->maSocket = 0;

Ca passe...

Merci pour votre aide

**Nasky** · 10/12/2005, 16h50

Et si tu fais ?

Code :

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

 data* pData = (data*)donnees;

Nas'

**sorry60** · 10/12/2005, 16h52

Envoyé par Nasky

Et si tu fais ?

Code :

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

 data* pData = (data*)donnees;

Nas'

Non ça ne change rien

**Médinoc** · 10/12/2005, 16h52

Ca risquera encore plus de bugguer comme ça (Les conversions explicites de pointeurs sont déconseillées en C quand on a des void * qui font ça très bien)

PS: Pourquoi tu appelles ta structure "pData" dans ton main ? Si c'est ainsi que tu nommes tes pointeurs, ça mènera à la confusion : dans le main, ce n'est pas un pointeur...

**Trap D** · 10/12/2005, 16h54

Ton pData, comment est-il intialisé exactement ?

Code :

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

1
2
3
4
5
6
7
8
9
10
nt main(int argc, char* argv[])
{
   /* Declaration des variables */
 
   data pData; //  <== à cet endroit il y a n 'importe quoi dedans.
 
   /*Connection du signal click de chaque bouton à la fonction appropriée*/    g_signal_connect(G_OBJECT(pData.pBtConnect),"clicked",G_CALLBACK(connection),&pData);
   /* ... */
   return 0;
}

**sorry60** · 10/12/2005, 16h55

Envoyé par Médinoc

Ca risquera encore plus de bugguer comme ça (Les conversions explicites de pointeurs sont déconseillées en C quand on a des void * qui font ça très bien)

PS: Pourquoi tu appelles ta structure "pData" dans ton main ? Si c'est ainsi que tu nommes tes pointeurs, ça mènera à la confusion : dans le main, ce n'est pas un pointeur...

Oui exacte le "p" est à retirer, c'est un oubli car au debut c'etait un pointeur

**sorry60** · 10/12/2005, 16h58

Envoyé par Trap D

Ton pData, comment est-il intialisé exactement ?

Ce n'est pas un pointeur.
Voila ce qu'il y a dedans :

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
/* Creation fenetre */
	creerFenetre(&pData.pFenetre,"MessInst",400,400);
	g_signal_connect(G_OBJECT(pData.pFenetre), "destroy", G_CALLBACK(gtk_main_quit), NULL);
	/* Ajout de la table dans la fenetre */
	gtk_container_add(GTK_CONTAINER(pData.pFenetre),GTK_WIDGET(pTable));
	/* Creations et ajouts des labels */
	creerAjouterLabel(&pData.pLbIp,"Entrez IP du serveur",pTable,0,1,0,1);
	creerAjouterLabel(&pData.pLbPort,"Entrez le port du serveur",pTable,0,1,2,3);
	creerAjouterLabel(&pData.pLbLogin,"Entrez votre login",pTable,1,2,0,1);
	creerAjouterLabel(&pData.pLbMdp,"Entrez votre mot de passe",pTable,1,2,2,3);
	creerAjouterLabel(&pData.pLbDest,"Entrez le destinataire",pTable,1,2,4,5);
	creerAjouterLabel(&pData.pLbMsg,"Entrez votre message",pTable,2,3,0,1);
	/* Creations et ajouts des zones de saisie */
	creerAjouterZoneSaisie(&pData.pZsIp,IP_SIZE_MAX,pTable,0,1,1,2);
	creerAjouterZoneSaisie(&pData.pZsPort,PORT_SIZE_MAX,pTable,0,1,3,4);
	creerAjouterZoneSaisie(&pData.pZsLogin,LOGIN_SIZE_MAX,pTable,1,2,1,2);
	creerAjouterZoneSaisie(&pData.pZsMdp,MDP_SIZE_MAX,pTable,1,2,3,4);
	creerAjouterZoneSaisie(&pData.pZsDest,LOGIN_SIZE_MAX,pTable,1,2,5,6);
	creerAjouterZoneSaisie(&pData.pZsMsg,MSG_SIZE_MAX,pTable,2,3,1,2);
	/* Creations et ajouts des boutons */
	creerAjouterBouton(&pData.pBtConnect,"Se connecter",pTable,0,1,4,5);
	creerAjouterBouton(&pData.pBtQuit,"Quitter",pTable,0,1,5,6);
	creerAjouterBouton(&pData.pBtSend,"Envoyer",pTable,2,3,2,3);
	/* Creation et ajout de la zone de texte */
	creerAjouterZoneText(&pData.pZtInfos,pTable,2,3,3,6);

creerAjouterLabel.c (le code pour les boutons, zones de saisie est quasiment identique) :

Code :

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

1
2
3
4
5
6
/* Fonction qui crée et ajoute un label dans une table*/
void creerAjouterLabel(GtkWidget** pLabel, const char* texte,GtkWidget *pTable,guint left, guint right, guint top, guint bottom)
{
	*pLabel = gtk_label_new(texte); /*création du label*/
	gtk_table_attach_defaults(GTK_TABLE(pTable),*pLabel,left,right,top,bottom);	/* Ajout dans la table */
}

**Nasky** · 10/12/2005, 16h58

Y'a un truc qui m'étonne avec le prototype de ta fonction connection qui est une callback.
Ce n'est pas supposé être :

Code :

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

void connection(GtkWidget *p, gpointer donnees);

A moins que ce ne soit pas obligatoire...
Mais en tout cas, si c'était ça, ça aurait été détecté à la compilation.

Je vois pas ton erreur... j'ai très souvent fait comme ça...

Nas'

**sorry60** · 10/12/2005, 17h00

Envoyé par Nasky

Y'a un truc qui m'étonne avec le prototype de ta fonction connection qui est une callback.
Ce n'est pas supposé être :

Code :

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

void connection(GtkWidget *p, gpointer donnees);

A moins que ce ne soit pas obligatoire...
Mais en tout cas, si c'était ça, ça aurait été détecté à la compilation.

Je vois pas ton erreur... j'ai très souvent fait comme ça...

Nas'

Oui c'est en effet le prototype type, mais comme je n'utilise pas le widget qui subit l'événement, je m'en passe dans le prototype.

J'ai aussi souvent fait comme ça, c'est pour ça que moi aussi je ne trouve pas l'erreur

**Médinoc** · 10/12/2005, 17h01

Eh bien, ne cherche plus ton erreur: Tu reçois le gtkwidget * à la place des données: Il faut que tu le mettes quand même!

**sorry60** · 10/12/2005, 17h02

Envoyé par Médinoc

Eh bien, ne cherche plus ton erreur: Tu reçois le gtkwidget * à la place des données: Il faut que tu le mettes quand même!

Ah ok.
Mais bon le fait que je n'utilise pas le widget par la suite dans la fonction me donne un warning traité en erreur vu que je compile avec l'option -Werror

Edit : c'etait bien ça l'erreur, comme quoi il ne fait jamais faire de zèle dans les prototypes !