Discussion :

[mjfcolas]

Bonjour,

J'ai réalisé un programme pour jouer au plus ou moins en réseau avec une interface graphique. Il compile, et il fonctionne dans l'absolu. Cependant, des segementation fault et autres problèmes de mémoires surviennent aléatoirement mais fréquemment. Notamment des "malloc(): memory corruption", des "*** glibc detected *** ./GPOM réseau: double free or corruption (fasttop): 0x09e4c2f8 ***" ou encore un "failed to allocate 3154956288 bytes" (Sans blagues...)

Un petit coup de debugger (que je maitrise bien peu) me renvoie souvent aux gtk_label_set_text, mais ce n'est pas le cas tout le temps...

Enfin bon, un problème que je n'arrive absolument pas à cerner...

Ci joint les codes du serveur et du client. Excusez la propreté du truc d'il vous plait

Merci d'avance, Emmanuel.

Serveur:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
#include <stdio.h>
#include <stdlib.h>
#include <gtk/gtk.h>
#include "define.h"
 
void* threadS(void* param)
{
    datasS *datas = param;
    GtkWidget *labelInfo = NULL, *labelClient = NULL;
    char messages[100]={0};
    char nombre[100] = {0};
 
    labelInfo = datas->labelInfo;
    labelClient = datas->labelClient;
 
    /*socket serveur*/
    SOCKET sock =  0;
    SOCKADDR_IN sin;
    socklen_t tailleserv = sizeof(sin);
 
    /*socket client*/
    SOCKET csock = 0;
    SOCKADDR_IN csin;
    socklen_t taillecli = sizeof(csin);
 
    int erreur = 0, nombreM = 0, nombreT = 0, comm = 0, gagne = 0, rejouer = 1;
 
    sock = socket(AF_INET, SOCK_STREAM, 0);
 
 
    if(sock != INVALID_SOCKET && sock != 0)
    {
        sprintf(messages,"La socket a été ouverte");
        gtk_label_set_text(GTK_LABEL(labelInfo), messages);
 
        sin.sin_addr.s_addr = htonl(INADDR_ANY);
        sin.sin_family = AF_INET;
        sin.sin_port = htons(20000);
        erreur = bind(sock, (SOCKADDR*)&sin, tailleserv);
        if(erreur != SOCKET_ERROR)
        {
            erreur = listen(sock, 5);
            sprintf(messages,"Connexion en écoute");
            gtk_label_set_text(GTK_LABEL(labelInfo), messages);
            if(erreur != SOCKET_ERROR)
            {
                sprintf(messages,"Attente de connexion d'un client");
                gtk_label_set_text(GTK_LABEL(labelInfo), messages);
                csock = accept(sock, (SOCKADDR*)&csin, &taillecli);
                sprintf(messages,"Un client s'est connecté");
                gtk_label_set_text(GTK_LABEL(labelInfo), messages);
            }
 
            /*Début du jeu*/
 
            sprintf(messages,"Attente du client");
            gtk_label_set_text(GTK_LABEL(labelClient), messages);
            recv(csock, &comm, sizeof(int), 0);
            if(comm == 0110)
            {
                comm = 1110;
                send(csock, &comm, sizeof(int), 0);
                sprintf(messages,"A vous de jouer: tapez le nombre mystère");
                gtk_label_set_text(GTK_LABEL(labelClient), messages);
                while(nombreM == 0)
                {
                    nombreM = datas->NM;
                }
                comm = 1111;
                send(csock, &comm, sizeof(int), 0);
                sprintf(messages,"Attendez...");
                gtk_label_set_text(GTK_LABEL(labelClient), messages);
 
                while(gagne == 0)
                {
                    recv(csock, &nombreT, sizeof(int), 0);
                    printf("SERVEUR: %d\n", nombreT);
 
                        if(nombreT < nombreM)
                        {
 
                            sprintf(messages, "%d: C'est plus!", nombreT);
                            gtk_label_set_text(GTK_LABEL(labelClient), messages);
                            comm = 1;
                            send(csock, &comm, sizeof(int), 0);
                        }
                        else if (nombreT > nombreM)
                        {
                            sprintf(messages, "%d: C'est moins!", nombreT);
                            gtk_label_set_text(GTK_LABEL(labelClient), messages);
                            comm = 2;
                            send(csock, &comm, sizeof(int), 0);
                        }
                        else
                        {
                            sprintf(messages, "L'adversaire à gagné!!");
                            gtk_label_set_text(GTK_LABEL(labelClient), messages);;
                            comm = 3;
                            send(csock, &comm, sizeof(int), 0);
                            gagne = 1;
                        }
 
                }
 
            }
            else
            {
            }
 
        }
 
    }
    sprintf(messages,"Fermeture des connexions");
    gtk_label_set_text(GTK_LABEL(labelInfo), messages);
    //shutdown(csock, 2);
    closesocket(sock);
    closesocket(csock);
 
}
 
void choisirNM(GtkWidget *widget, gpointer *struc)
{
    datasS *datas = NULL;
    datas = struc;
    GtkWidget *entry = NULL;
    const gchar *txt = NULL;
 
    entry = datas->entryNombre;
 
    txt = gtk_entry_get_text(GTK_ENTRY(entry));
    datas->NM = atoi(txt);
 
}
 
 
void serveur(GtkWidget *widget, gpointer *windowMain)
{
 
    #if defined (WIN32)
        WSADATA WSAData;
        WSAStartup(MAKEWORD(2,2), &WSAData);
    #endif
 
    GtkWidget *window = windowMain, *table = NULL, *bCredit = NULL, *bQuitter = NULL, *bOk = NULL;
    GtkWidget *entryNombre = NULL, *labelInfo = NULL, *labelClient = NULL;
 
    /*structure pour le thread*/
    datasS *datas = NULL;
    int tailleMemoire = 0;
    datas = malloc(sizeof(datasS));
 
    /*Thread*/
    pthread_t thread = 0;
 
    /*On quitte la boucle du menu*/
    gtk_main_quit();
 
    window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
    gtk_window_set_position(GTK_WINDOW(window), GTK_WIN_POS_CENTER);
    gtk_window_set_title(GTK_WINDOW(window), "Serveur POM");
    gtk_window_set_default_size(GTK_WINDOW(window), 400, 200);
 
    table = gtk_table_new(4, 2, TRUE);
    gtk_container_add(GTK_CONTAINER(window), table);
 
    bCredit  = creerBouton(table, "A propos", 1, 2, 3, 4);
    bQuitter = creerBouton(table, "Quitter" , 0, 1, 3, 4);
    bOk      = creerBouton(table, "OK!     ", 1, 2, 1, 2);
    labelInfo = creerLabel(table, "Info Connexion", 0, 2, 0, 1);
    labelClient = creerLabel(table, "Info Client", 0, 2, 2, 3);
    entryNombre = creerEntry(table, "", 0, 1, 1, 2);
 
    datas->labelInfo = labelInfo;
    datas->labelClient = labelClient;
    datas->entryNombre = entryNombre;
    datas->NM = 0;
 
    pthread_create(&thread, NULL, threadS, (void*)datas);
 
    gtk_widget_show_all(window);
 
    g_signal_connect(G_OBJECT(window), "destroy", G_CALLBACK(gtk_main_quit), NULL);
    g_signal_connect(G_OBJECT(bQuitter), "clicked", G_CALLBACK(gtk_main_quit), NULL);
    g_signal_connect(G_OBJECT(bOk), "clicked", G_CALLBACK(choisirNM), datas);
 
    gtk_main();
 
    free(datas);
 
    #if defined (WIN32)
        WSACleanup();
    #endif
 
 
}

Client:

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
#include <stdio.h>
#include <stdlib.h>
#include <gtk/gtk.h>
#include "define.h"
 
void* threadC(void* param)
{
    datasC *datas = param;
    GtkWidget *labelInfo = NULL, *labelPOM = NULL;
    char messages[100]={0};
 
    labelInfo = datas->labelInfo;
    labelPOM = datas->labelPOM;
 
    /* socket client */
    SOCKET sock;
    SOCKADDR_IN sin;
    socklen_t taille = sizeof(sin);
 
    int connexion = 0;
    int nombreT = 0, comm = 0, gagne = 0, rejouer = 1, i = 0;
    char IP[14] = {0};
 
    sock = socket(AF_INET, SOCK_STREAM, 0);
 
    if(sock != INVALID_SOCKET)
    {
        sprintf(messages,"La socket a été ouverte");
        gtk_label_set_text(GTK_LABEL(labelInfo), messages);
        sprintf(messages,"Entrez l'adresse IP à laquelle vous voulez vous conecter");
        gtk_label_set_text(GTK_LABEL(labelInfo), messages);
 
        while(IP[0] == 0)
        {
            for(i = 0; i<=14; i++)
            {
            IP[i] = datas->IP[i];
            }
        }
 
        sin.sin_addr.s_addr = inet_addr(IP);
        sin.sin_family = AF_INET;
        sin.sin_port = htons(20000);
 
        sprintf(messages,"Connexion au serveur...");
        gtk_label_set_text(GTK_LABEL(labelInfo), messages);
 
        do
        {
        connexion = connect(sock, (SOCKADDR*)&sin, taille);
        }
        while(connexion != 0);
 
        sprintf(messages,"La connexion a réussie");
        gtk_label_set_text(GTK_LABEL(labelInfo), messages);
 
        /*Debut du jeu*/
 
        comm = 0110;
        send(sock, &comm, sizeof(int), 0);
        sprintf(messages,"Attente du serveur...");
        gtk_label_set_text(GTK_LABEL(labelInfo), messages);
        recv(sock, &comm, sizeof(int), 0);
        if(comm == 1110)
        {
            sprintf(messages,"L'adversaire choisi un nombre mystère, attendez...");
            gtk_label_set_text(GTK_LABEL(labelInfo), messages);
            recv(sock, &comm, sizeof(int), 0);
            sprintf(messages,"Le nombre mystère a été choisi");
            gtk_label_set_text(GTK_LABEL(labelInfo), messages);
 
            while(gagne == 0)
            {
                sprintf(messages,"Quel est le nombre mystère?");
                gtk_label_set_text(GTK_LABEL(labelInfo), messages);
                while(nombreT == 0)
                {
                    nombreT = datas->NT;
                }
                send(sock, &nombreT, sizeof(int), 0);
                printf("TEST: %d\n", nombreT);
                recv(sock, &comm, sizeof(int), 0);
                if(comm == 1)
                {
                    sprintf(messages,"%d: C'est Plus!!", nombreT);
                    gtk_label_set_text(GTK_LABEL(labelPOM), messages);
                }
                else if(comm == 2)
                {
                    sprintf(messages,"%d: C'est Moins!!", nombreT);
                    gtk_label_set_text(GTK_LABEL(labelPOM), messages);
                }
                else if(comm == 3)
                {
                    sprintf(messages,"%d: Vous avez gagné!!", nombreT);
                    gtk_label_set_text(GTK_LABEL(labelPOM), messages);
                    gagne = 1;
                }
                nombreT = 0;
                datas->NT = 0;
 
            }
 
 
        }
        else
        {
        }
 
 
    }
    sprintf(messages,"Fermeture des connexions");
    gtk_label_set_text(GTK_LABEL(labelInfo), messages);
    //shutdown(sock, 2);
    closesocket(sock);
 
}
 
void choisirNT(GtkWidget *widget, gpointer *struc)
{
    datasC *datas = NULL;
    datas = struc;
    GtkWidget *entry = NULL;
    const gchar *txt = NULL;
 
    entry = datas->entryNombre;
 
    txt = gtk_entry_get_text(GTK_ENTRY(entry));
    datas->NT = atoi(txt);
 
}
 
void enregistrerIP(GtkWidget *widget, gpointer *struc)
{
    datasC *datas = NULL;
    datas = struc;
    GtkWidget *entry = NULL;
    const gchar *IP = NULL;
    int i = 0;
 
    entry = datas->entryIP;
 
    IP = gtk_entry_get_text(GTK_ENTRY(entry));
    for( i = 0; i<= 15; i++)
    {
        datas->IP[i] = IP[i];
    }
}
 
void client(GtkWidget *widget, gpointer *windowMain)
{
 
    #if defined (WIN32)
        WSADATA WSAData;
        WSAStartup(MAKEWORD(2,2), &WSAData);
    #endif
 
    GtkWidget *window = windowMain, *table = NULL, *bCredit = NULL, *bQuitter = NULL, *bOk = NULL, *bCo = NULL;
    GtkWidget *entryNombre = NULL, *entryIP = NULL, *labelPOM = NULL, *labelInfo = NULL;
 
    int i = 0;
 
    datasC *datas = NULL;
    datas = malloc(sizeof(datasC));
 
    pthread_t thread = 0;
 
    /*On quitte la boucle du menu*/
    gtk_main_quit();
 
    window = gtk_window_new(GTK_WINDOW_TOPLEVEL);
    gtk_window_set_position(GTK_WINDOW(window), GTK_WIN_POS_CENTER);
    gtk_window_set_title(GTK_WINDOW(window), "Client POM");
    gtk_window_set_default_size(GTK_WINDOW(window), 400, 300);
 
    table = gtk_table_new(5, 2, TRUE);
    gtk_container_add(GTK_CONTAINER(window), table);
 
    bCredit  = creerBouton(table, "A propos", 1, 2, 4, 5);
    bQuitter = creerBouton(table, "Quitter" , 0, 1, 4, 5);
    bOk      = creerBouton(table, "OK!", 1, 2, 3, 4);
    bCo      = creerBouton(table, "Connexion", 1, 2, 1, 2);
    labelInfo= creerLabel(table, "Info Connexion", 0, 2, 0, 1);
    labelPOM = creerLabel(table, "Plus ou Moins", 0, 1, 2, 3);
    entryIP  = creerEntry(table, "Entrez l'Ip ici", 0, 1, 1, 2);
    entryNombre = creerEntry(table, "", 1, 2, 2, 3);
 
    datas->entryIP = entryIP;
    datas->labelInfo = labelInfo;
    for(i = 0; i <= 14; i++)
    {
        datas->IP[i] = 0;
    }
    datas->NT = 0;
    datas->entryNombre = entryNombre;
    datas->labelPOM = labelPOM;
 
 
    pthread_create(&thread, NULL, threadC, (void*)datas);
 
    gtk_widget_show_all(window);
 
    g_signal_connect(G_OBJECT(window), "destroy", G_CALLBACK(gtk_main_quit), NULL);
    g_signal_connect(G_OBJECT(bQuitter), "clicked", G_CALLBACK(gtk_main_quit), NULL);
    g_signal_connect(G_OBJECT(bCo), "clicked", G_CALLBACK(enregistrerIP), datas);
    g_signal_connect(G_OBJECT(bOk), "clicked", G_CALLBACK(choisirNT), datas);
 
    gtk_main();
 
    free(datas);
 
    #if defined (WIN32)
        WSACleanup();
    #endif
 
}

[Invité]

Je ne connais rien aux threads, mais à tout hasard :
- remplace tes malloc par une déclaration sans pointeur : (dataS data, envoies ta structure en tant que pointeur quand il faut, et enlèves tes free.
- quand tu utilises pthreads_create tu castes la structure, mais en arrivant dans la fonction thread tu ne castes pas dans l'autre sens (je ne sais pas si c'est vraiment important)
- dans la fonction choisirNM tu reçois en paramètre un élément *struc de type gpointer mais tu l'associes ensuite à datas qui est de type dataS, c'est normal ?