problème chat avec socket/fork/pipe

**cedricdd** · 08/03/2007, 02h07

Voila j'ai comme projet de réaliser un chat en C,en utilisant les sockets, fork et des pipes, mais j'ai des erreurs dans mon serveur, pourtant je ne vois pas ou j'ai faux

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
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <sys/socket.h>
#include <sys/types.h>
#include <netinet/in.h>
#include <fcntl.h>
#include <unistd.h>
#define maxclient 10
 
 
typedef struct{
	int envoie[2];
	int recep[2];
	int libre;
}t_fils;
 
int main()
 
{
	system("clear");
	printf("Lancement du serveur\n\n");
 
	t_fils tabFils[maxclient];
	int i=0,nbrclient=0,client=0,traite=1,test,test2,sock,sock2,duplication,placeclient=0,clientok=0,placeclientenvoie;
	int sizet= sizeof(struct sockaddr);
	struct sockaddr_in sin,csin;
	char bufferR[1024]="",bufferE[1024]="",buffer[1024]="",temp[10]="";
 
	if ( (sock=socket ( AF_INET, SOCK_STREAM, 6)) == -1)
	{
		printf("Socket creation erreur");
	}
 
 
	memset ( &sin, 0, sizeof(struct sockaddr));
	memset ( &csin, 0, sizeof(struct sockaddr));
 
	sin.sin_family= AF_INET;
	sin.sin_port=htons( 6222);				
	sin.sin_addr.s_addr=htonl(INADDR_ANY);
 
 
	if ( bind( sock,(struct sockaddr *)&sin, sizeof(struct sockaddr) ) == -1 )
	{
		printf("Impossible lier socket");
	}
	else
	{							
		printf("Serveur ok, en attente de client, nombre maximun de client %i\n\n",maxclient);
	}	
 
	listen(sock, 1);
 
	for(i=0;i<maxclient;i++)
		{
			tabFils[i].libre=0;
		}
 
 
	fcntl(sock, F_SETFL, fcntl ( sock, F_GETFL ) | O_NONBLOCK);
 
	while(nbrclient<10)		//Boucle plusieurs clients
	{
 
 
		client=0;
 
 
 
 
	   	if( (sock2 = accept(sock ,(struct sockaddr *)&csin, (socklen_t *)&sizet ) ) == -1 )
		{
			//printf("Impossible d'accepter");
		}
	    	else 
		{
			printf("\nCLIENT ENTRANT\n\n");
			client=1;
		}
 
 
		if ( client == 1)
		{
			nbrclient+=1; //Un nouveau client
 
 
			do	//Le nouveau client prendra la premiere place libre
			{
				if(tabFils[placeclient].libre == 0) //La place est libre
				{
					if (  pipe(tabFils[placeclient].recep) == -1 || pipe(tabFils[placeclient].envoie) == -1 ) 	//Pour communiquer entre le pere et le fils
					{
						printf("Erreur lors de la déclaration des tubes");
					}
					clientok=1;
				}
				else	//Un client occupe déjà cette place
				{
					placeclient++;
				}
			}while(clientok !=1);	//Le client à trouvé une place
 
 
			duplication=fork();
 
			if( duplication == -1)
			{
				printf("Erreur dans la duplication du processus");
				exit(0);
			}
 
			if( duplication !=0)  //Processus pere
			{
				close(sock2);
				close(tabFils[placeclient].envoie[0]);
				close(tabFils[placeclient].recep[1]);
				fcntl(tabFils[placeclient].recep[0], F_SETFL, fcntl (tabFils[placeclient].recep[0], F_GETFL ) | O_NONBLOCK);
				tabFils[placeclient].libre=1;
			}
 
			else	// Processus fils
			{
 
				close(tabFils[placeclient].envoie[1]);
				close(tabFils[placeclient].recep[0]);
 
				fcntl(sock2, F_SETFL, fcntl ( sock2, F_GETFL ) | O_NONBLOCK);
				fcntl(tabFils[placeclient].envoie[0], F_SETFL, fcntl (tabFils[placeclient].envoie[0],  F_GETFL ) | O_NONBLOCK);
 
 
				while(1)
				{
 
 
					memset(bufferR, 0, sizeof(bufferR));
					memset(buffer, 0, sizeof(buffer));
 
					if ( read(sock2, bufferR, 255) > 0 )
					{
						if ( strlen( bufferR ) !=0) //Pour eviter les messages fantomes
						{
							strncpy(temp, bufferR, 4);
 
							if( strcmp( temp, "exit") == 0 )
							{
								write(tabFils[placeclient].recep[1], temp, sizeof(temp));
								close(tabFils[placeclient].envoie[0]);
								close(tabFils[placeclient].recep[1]);
							}
							else
							{
 
								write(tabFils[placeclient].recep[1], bufferR, sizeof(bufferR));
 
							}
						}					
					}
 
 
					memset(bufferE, 0, sizeof(bufferE));
					if ( read (tabFils[placeclient].envoie[0],  bufferE, 1024) > 0 )
					{
						if ( strlen ( bufferE ) != 0 )
						{
							write(sock2, bufferE, sizeof(bufferE));
							printf("J'envoie au client %s",bufferE);
						}
					}
 
				} 
 
			} 		//Fin du fils
 
 
 
		}
									//Traitement dans le père
 
				for(test=0;test<nbrclient;test++)
				{
					if( read( tabFils[test].recep[0], bufferR, 1024) > 0)
					{
						if( strlen ( bufferR) !=0 )
						{
 
 
							placeclientenvoie=test;
 
							strncpy(temp, bufferR, 4);
							if( strcmp( temp, "exit") == 0 )
							{
								printf("Un client vient de partir, il en reste %i\n",nbrclient-1);
								traite=0;
								nbrclient--;
								tabFils[test].libre=0;
							}
 
 
							if ( traite == 1)	//Si se n'est un message de deconnection
							{
								printf("le pere affiche:  %s\n",bufferR);
								if( nbrclient==1)
								{
									write(tabFils[0].envoie[1], "Vous êtes le seul client\r\n", sizeof("Vous êtes le seul client\r\n"));
								}
 
								else
								{
									for(test2=0;test<nbrclient;test2++)
									{
										if(placeclientenvoie != test2)	//On ne renvoie pas a celui qui vient d'écrire
										{
											write(tabFils[test2].envoie[1], bufferR, sizeof(bufferR));
										}
									}
								}
							}
 
							else	//De base on retraite le suivant
							{
								traite=1;
							}
						}
					}
				}
 
 
 
 
	}  		//Fin boucle plusieurs clients
 
 
close(sock);
close(sock2);	
 
}

**cedricdd** · 08/03/2007, 07h15

voila j'ai mis en commentaire une partie du programme pour n'avoir qu'un client qui envoie un message avec le processus fils qui recoit le message et qui le transmet au père qui lui va juste le renvoier au fils qui va le renvoier au client, juste pour tester, y a pas de problème jusqu'a que le père renvoie le message, le premier message arrivera au client mais en même temps y a une boucle d'affichage du message dans le serveur qui se lance, et va occupé toute la memoire, je trouve pas de quoi ca provient

**Médinoc** · 08/03/2007, 09h25

C'est [ CODE ] (

) et pas [ QUOTE ] (

) qu'il faut utiliser...
Je ne pense pas qu'un serveur-fork soit le meilleur choix pour un chat : En effet, pas facile lorsqu'un nouveau client se connecte, de s'assurer que lui aussi reçoive les messages des clients déjà connectés.
Tu devrais faire un serveur en un seul processus et un seul thread, en utilisant select() pour lire sur les sockets...

**Médinoc** · 08/03/2007, 09h29

PS: C'est vraiment bizarre qu'un des processus se mette à occuper toute la mémoire puisqu'il n'y a aucune allocation ni aucun appel de fonction récursif (d'ailleurs, tu as tout mis dans le main(), c'est du goret force 10...)

Es-tu sûr que ce ne sont pas plutôt les processus qui prolifèrent (exces de fork()) ?

**cedricdd** · 08/03/2007, 09h30

Je me suis rendu compe que c'est super chiant comme ca, mais c'est imposser dans le sujet.

Non y a pas de nouveau processus qui se lance quand ca part en couille

**Médinoc** · 08/03/2007, 09h33

Qu'est-ce qui est imposé ? Tout le code dans le main ou le serveur-fork ?

Si c'est tout le code dans le main, change de prof.
Si c'est le serveur-fork, tu auras besoin de mémoire partagée (au minimum) pour que ce soit un chat (un serveur-fork ne peut fonctionner sans s'il y a une interaction entre les clients). Peut-être serait-il intéressant de le rappeler au prof, voire d'en changer aussi s'il ne comprend pas...

Edit: Ah, j'ai compris!
Le serveur-fork communique avec tous les autres par pipes, comme s'il était lui-même un serveur!
Génial, un serveur à deux niveaux!

Edit2: Dans tous les cas, il te faudra un select(). Ici, il sera sur les pipes.

PS:

est ton ami.