Segmentation fault incomprehensible

Version imprimable

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:

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:

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:

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:

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

Surtout que quand je fais :

Code:

pData->maSocket = 0;

Ca passe... :?

Merci pour votre aide

10/12/2005, 16h50
Nasky

Et si tu fais ?

Code:

data* pData = (data*)donnees;

Nas'
10/12/2005, 16h52
sorry60

Citation:

Envoyé par Nasky

Et si tu fais ?

Code:

data* pData = (data*)donnees;

Nas'

Non ça ne change rien :(
10/12/2005, 16h52
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...

Ton pData, comment est-il intialisé exactement ?

Code:

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;
}

10/12/2005, 16h55
sorry60

Citation:

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 :oops:

Citation:

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:

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:

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 */
}

10/12/2005, 16h58
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:

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'
10/12/2005, 17h00
sorry60

Citation:

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:

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 8O
10/12/2005, 17h01
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!
10/12/2005, 17h02
sorry60

Citation:

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 !
10/12/2005, 17h05
Médinoc

void connection(GtkWidget *p, gpointer donnees)
{
(void)p;
//blablabla
}
10/12/2005, 17h06
sorry60

Citation:

Envoyé par Médinoc

void connection(GtkWidget *p, gpointer donnees)
{
(void)p;
//blablabla
}

Ok :lol:

Merci pour ton aide :wink: