Discussion :

[Kronoob]

Bonsoir à tous,

Alors je doit réaliser un projet de serveur-client qui doit entre autre exécuter des script bash et envoyer des infos via un socket.

J'ai (avec l'aide de certains ici présent) aboutis à ce code (qui est surement perfectible, mais le temps m'est compté).

Code c :

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
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
#include <stdio.h>
#include <stdlib.h>
#include <sys/socket.h>
#include <sys/wait.h>
#include <sys/types.h>
#include <netinet/in.h>
#include <string.h>
#include <unistd.h>
#include <errno.h>
 
#define Port 9100
#define AdresseIP "127.0.0.1"
 
#define TailleChaine 255
 
#define ArretServeur 0
#define ArretClient 1
#define Fermeture 2
#define Commande1 3
#define Commande2 4
#define	Commande3 5
#define ErreurCommande 6
#define MauvaiseCommande 7
 
#define Script1 "./ListeCommandeProcessus"
#define Script2 "./"
#define Script3 "./"
 
int LectureChoix(int Socket)
{
	int Recus, Test = 0;
 
	do
	{
		if(read(Socket, &Recus, sizeof(int)) > 0);
			return(Recus);
	}while(++Test<1000);
 
	return(-1);
}
 
int EnvoisStatus(int Socket, int Message)
{
	int Test = 0;
 
	do
	{
		if(write(Socket, &Message, sizeof(int)) > 0)
		{
			return(1);
		}
	}while(++Test<1000);
 
	return(-1);
}
 
int EnvoisFichier(int Socket, char *NomFichier)
{
	FILE *Fichier;
	char Chaine[TailleChaine];
	int IndiceChaine;
 
	//ouverture de fichier1
	Fichier = fopen(NomFichier, "r");
 
	//test si fichier1 existe
	if (Fichier == NULL)
	{
		//envois du status ErreurCommande
		if(EnvoisStatus(Socket, ErreurCommande) == -1)
		{
			//erreur d'écriture
			return(-1);
		}
	}
	else
	{
		//envois du fichier1 tant qu'on est pas à la fin
		while(!eof(Fichier))
		{
			//initialisation de l'indice
			IndiceChaine = 0;
 
			do
			{
				//lecture d'un caractère
				Chaine[IndiceChaine] = getc(Fichier);
 
			}while(Chaine[IndiceChaine] != EOF && IndiceChaine++ <= TailleChaine);
 
			//envois dans la chaussette
			if(write(Socket, Chaine, TailleChaine) <= 0)
			{
				//erreur d'écriture
				return(-1);
			}
 
		}
 
		//ferme fichier1
		fclose(Fichier);
	}
}
 
int Execution(int Socket, int NumCommande)
{
	int PIDfork, StatusFils;
 
	//duplication du processus
	PIDfork = fork();
 
	//qui suis-je ?
	if (PIDfork != 0)
	{
		//Processus Parent
		wait(&StatusFils);
 
		if(!WIFEXITED(StatusFils))
		{
			//envois du status ErreurCommande
			if(EnvoisStatus(Socket, ErreurCommande) == -1)
			{
				//erreur d'écriture
				printf("erreur sur EnvoisStatus\n");
				return(-1);
			}
		}
		else
		{
			//envois du status NumCommande
			if(EnvoisStatus(Socket, NumCommande) == -1)
			{
				//erreur d'écriture
				printf("erreur sur EnvoisStatus\n");
				return(-1);
			}
 
			//envois du fichier1
			if(EnvoisFichier(Socket, "fichier1") == -1)
			{
				//erreur d'envois de fichier
				printf("erreur sur EnvoisFichier\n");
				return(-1);
			}
		}
	}
	else
	{
		//Processus Fils
 
		//choix du script
		switch(NumCommande)
		{
			case Commande1 :
				//execution du script 1
				exec(Script1);
 
				break;
 
			case Commande2 :
				//execution du script 1
				exec(Script2);
 
				break;
 
			case Commande3 :
				//execution du script 1
				exec(Script3);
 
				break;
		}
 
		un_exit(EXIT_FAILURE);
	}
}
 
int main()
{
	//déclaration de la socket
	int SocketConnexion, SocketCommunication;
	struct sockaddr_in SocketConnexionParam, SocketCommunicationParam;
 
	//déclaration des variables de contrôle
	int Choix, Bouclage;
 
	//initialisation des variables
	errno=0;
	Bouclage = 1;
 
	//déclaration de la taille du socket de communication
	socklen_t SocketCommunicationParamLong;
 
	//création de la socket de connexion
	SocketConnexion = socket(PF_INET, SOCK_STREAM, 0);
	if(SocketConnexion == -1)
	{
		printf("erreur sur socket : %s\n", strerror(errno));
		return(1);
	}
 
	//initialisation de l'adresse de connexion
	bzero(&SocketConnexionParam, sizeof(SocketConnexionParam));
	SocketConnexionParam.sin_family = PF_INET;
	SocketConnexionParam.sin_port = htons(Port);
	SocketConnexionParam.sin_addr.s_addr = inet_addr(AdresseIP);
 
	//liaison entre le nom et la socket de connexion
	if(bind(SocketConnexion, (struct sockaddr *) &SocketConnexionParam, sizeof(SocketConnexionParam)) != 0)
	{
		printf("erreur sur bind : %s\n", strerror(errno));
		close(SocketConnexion);
		return(1);
	}
 
	//mise en écoute de la socket de connexion
	if(listen(SocketConnexion, 10) != 0)
	{
		printf("erreur sur listen : %s\n", strerror(errno));
		close(SocketConnexion);
		return(1);
	}
 
	while(1)
	{
		//accepte les connexions entrantes
		SocketCommunicationParamLong = sizeof(SocketCommunicationParam);
		SocketCommunication = accept(SocketConnexion, (struct sockaddr *) &SocketCommunicationParam, &SocketCommunicationParamLong);
 
		//test si l'accept est bien passé
		if(SocketCommunication != -1)
		{
			//boucle tant que bouclage = 1
			while(Bouclage)
			{
				//lecture du choix
				Choix = LectureChoix(SocketCommunication);
 
				//test si bonne lecture
				if(Choix == -1);
				{
					//erreur de lecture
					printf("erreur sur LectureChoix\n");
					return(1);
				}
 
				//choisi la bonne action suivant choix
				switch(Choix)
				{
					//choix d'arret du serveur
					case ArretServeur :
						close(SocketCommunication);
						close(SocketConnexion);
						return(0);
 
					//choix d'arret de la connection - sortie de la boucle while
					case Fermeture :
						close(SocketCommunication);
						Bouclage = 0;
						break;
 
					//choix d'execution du 1er script
					case Commande1 :
						//lancement d'execution
						if(Execution(SocketCommunication, Commande1) == -1)
						{
							printf("erreur sur Execution\n");
							return(1);
						}
 
						break;
 
					//choix d'execution du 1er script
					case Commande2 :
						//lancement d'execution
						if(Execution(SocketCommunication, Commande2) == -1)
						{
							printf("erreur sur Execution\n");
							return(1);
						}
 
						break;
 
					//choix d'execution du 1er script
					case Commande3 :
						//lancement d'execution
						if(Execution(SocketCommunication, Commande3) == -1)
						{
							printf("erreur sur Execution\n");
							return(1);
						}
 
 
						break;
 
					default :
						//envois du status MauvaiseCommande
						if(EnvoisStatus(SocketCommunication, MauvaiseCommande) == -1)
						{
							//erreur d'écriture
							printf("erreur sur EnvoisStatus\n");
							return(1);
						}
						break;
 
				}//fin choix
			}//fin while(Bouclage)
		}
		else
		{
			//erreur sur accept()
			printf("erreur sur accept : %s\n", strerror(errno));
			close(SocketConnexion);
			return(1);
		}
	}//end while(1)
}

Mais à la compilation j'obtiens ceci :

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
[titi@toto ProjetLinux]$ gcc -Wall -o Serveur Serveur.c 
Serveur.c: In function 'EnvoisFichier': 
Serveur.c:79: attention : implicit declaration of function 'eof' 
Serveur.c:100: attention : control reaches end of non-void function 
Serveur.c: In function 'Execution': 
Serveur.c:153: attention : implicit declaration of function 'exec' 
Serveur.c:170: attention : implicit declaration of function 'un_exit' 
Serveur.c:172: attention : control reaches end of non-void function 
Serveur.c: In function 'main': 
Serveur.c:202: attention : implicit declaration of function 'inet_addr' 
/tmp/cc3G3ru2.o: In function `EnvoisFichier': 
Serveur.c:(.text+0x13c): undefined reference to `eof' 
/tmp/cc3G3ru2.o: In function `Execution': 
Serveur.c:(.text+0x25a): undefined reference to `exec' 
Serveur.c:(.text+0x268): undefined reference to `exec' 
Serveur.c:(.text+0x276): undefined reference to `exec' 
Serveur.c:(.text+0x282): undefined reference to `un_exit' 
collect2: ld a retourné 1 code d'état d'exécution 
[titi@toto ProjetLinux]$

Il y a donc manifestement un problème à la compilation. apparement avec les fichier objet (si ma mémoire et bonne, les fichier .o sont appelé objet).

Mais, et j'en arrive à ma question, pourquoi ces erreurs ?

Si une âme charitable pouvait m'aider. Merci

[publicStaticVoidMain]

Mais à la compilation j'obtiens ceci :

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
[titi@toto ProjetLinux]$ gcc -Wall -o Serveur Serveur.c 
Serveur.c: In function 'EnvoisFichier': 
Serveur.c:79: attention : implicit declaration of function 'eof' 
Serveur.c:100: attention : control reaches end of non-void function 
Serveur.c: In function 'Execution': 
Serveur.c:153: attention : implicit declaration of function 'exec' 
Serveur.c:170: attention : implicit declaration of function 'un_exit' 
Serveur.c:172: attention : control reaches end of non-void function 
Serveur.c: In function 'main': 
Serveur.c:202: attention : implicit declaration of function 'inet_addr' 
/tmp/cc3G3ru2.o: In function `EnvoisFichier': 
Serveur.c:(.text+0x13c): undefined reference to `eof' 
/tmp/cc3G3ru2.o: In function `Execution': 
Serveur.c:(.text+0x25a): undefined reference to `exec' 
Serveur.c:(.text+0x268): undefined reference to `exec' 
Serveur.c:(.text+0x276): undefined reference to `exec' 
Serveur.c:(.text+0x282): undefined reference to `un_exit' 
collect2: ld a retourné 1 code d'état d'exécution 
[titi@toto ProjetLinux]$

Bah les erreurs sont bien décrites non :
Ligne 79 : la fonction eof n'existe pas. C'est feof()
Ligne 100 : T'as oublié un return dans ta fonction EnvoisFichier.
Ligne 153 : Je ne savais pas ce que t'avais en tete mais la fonction exec existe déjà. Bah la il y'a 2 choses, soit c'est une fonction que t'as déjà définie qui s'appelle exec donc là il y'aura conflit avec la fonction exec de unistd.h ou tu voulais la fonction exec mais tu lui passes de mauvais paramètres. Comme je vois t'es sur linux, un man exec devrait t'aider.
Ligne 172 : un_exit? connait pas. A toi de corriger ca !!
Ligne 202 : a la place de inet_addr, utilise htonl(INADDR_ANY);

Ce serait bien que tu lises un tutoriel sur les sockets en C. Il en existe bien sur sur dvp .
A++

[Kronoob]

Sauf que justement, c'est bien écris. Ce ne sont pas des erreurs, mais des attentions (warning). Juste parce que j'utilise l'option -Wall. Sans cette option les

implicit declaration of function

n'apparaissent pas.

Donc j'en conclus qu'il n'y a pas de problème à la compilation. Et pour ton info, les eof et autre un_exit mon été indiqués par Anomaly, qui est loin d'etre une quiche en prog système sous linux. Je pars donc du principe que le problème n'est pas là.

Par contre je comprends pas pourquoi les erreurs sont au niveau des fichiers objets (.o)

Par contre je suis d'accord, il manque un return sur EnvoisFichier.

Pour exec, c'est tiré d'un tuto de dvp. Je regarde le man.


EDIT :
Bon j'ai changer mes exec() par des execv() et mon un_exit() par un exit().
J'ai dû mal comprendre ce que ma dit Anomaly

Code c :

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
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
#include <stdio.h>
#include <stdlib.h>
#include <sys/socket.h>
#include <sys/wait.h>
#include <sys/types.h>
#include <netinet/in.h>
#include <string.h>
#include <unistd.h>
#include <errno.h>
 
#define Port 9100
#define AdresseIP "127.0.0.1"
 
#define TailleChaine 255
 
#define ArretServeur 0
#define ArretClient 1
#define Fermeture 2
#define Commande1 3
#define Commande2 4
#define	Commande3 5
#define ErreurCommande 6
#define MauvaiseCommande 7
 
#define Script1 "./ListeCommandeProcessus"
#define Script2 "./"
#define Script3 "./"
 
int LectureChoix(int Socket)
{
	int Recus, Test = 0;
 
	do
	{
		if(read(Socket, &Recus, sizeof(int)) > 0);
			return(Recus);
	}while(++Test<1000);
 
	return(-1);
}
 
int EnvoisStatus(int Socket, int Message)
{
	int Test = 0;
 
	do
	{
		if(write(Socket, &Message, sizeof(int)) > 0)
		{
			return(1);
		}
	}while(++Test<1000);
 
	return(-1);
}
 
int EnvoisFichier(int Socket, char *NomFichier)
{
	FILE *Fichier;
	char Chaine[TailleChaine];
	int IndiceChaine;
 
	//ouverture de fichier1
	Fichier = fopen(NomFichier, "r");
 
	//test si fichier1 existe
	if (Fichier == NULL)
	{
		//envois du status ErreurCommande
		if(EnvoisStatus(Socket, ErreurCommande) == -1)
		{
			//erreur d'écriture
			return(-1);
		}
	}
	else
	{
		//envois du fichier1 tant qu'on est pas à la fin
		while(!eof(Fichier))
		{
			//initialisation de l'indice
			IndiceChaine = 0;
 
			do
			{
				//lecture d'un caractère
				Chaine[IndiceChaine] = getc(Fichier);
 
			}while(Chaine[IndiceChaine] != EOF && IndiceChaine++ <= TailleChaine);
 
			//envois dans la chaussette
			if(write(Socket, Chaine, TailleChaine) <= 0)
			{
				//erreur d'écriture
				return(-1);
			}
 
		}
 
		//ferme fichier1
		fclose(Fichier);
	}
}
 
int Execution(int Socket, int NumCommande)
{
	int PIDfork, StatusFils;
	char* ArgumentCommande1[] = {Script1, NULL};
	char* ArgumentCommande2[] = {Script2, NULL};
	char* ArgumentCommande3[] = {Script3, NULL};
 
	//duplication du processus
	PIDfork = fork();
 
	//qui suis-je ?
	if (PIDfork != 0)
	{
		//Processus Parent
		wait(&StatusFils);
 
		if(!WIFEXITED(StatusFils))
		{
			//envois du status ErreurCommande
			if(EnvoisStatus(Socket, ErreurCommande) == -1)
			{
				//erreur d'écriture
				printf("erreur sur EnvoisStatus\n");
				return(-1);
			}
		}
		else
		{
			//envois du status NumCommande
			if(EnvoisStatus(Socket, NumCommande) == -1)
			{
				//erreur d'écriture
				printf("erreur sur EnvoisStatus\n");
				return(-1);
			}
 
			//envois du fichier1
			if(EnvoisFichier(Socket, "fichier1") == -1)
			{
				//erreur d'envois de fichier
				printf("erreur sur EnvoisFichier\n");
				return(-1);
			}
		}
	}
	else
	{
		//Processus Fils
 
		//choix du script
		switch(NumCommande)
		{
			case Commande1 :
				//execution du script 1
				execv(Script1, ArgumentCommande1);
 
				break;
 
			case Commande2 :
				//execution du script 1
				execv(Script2, ArgumentCommande2);
 
				break;
 
			case Commande3 :
				//execution du script 1
				exec(Script3, ArgumentCommande3);
 
				break;
		}
 
		exit(EXIT_FAILURE);
	}
}
 
int main()
{
	//déclaration de la socket
	int SocketConnexion, SocketCommunication;
	struct sockaddr_in SocketConnexionParam, SocketCommunicationParam;
 
	//déclaration des variables de contrôle
	int Choix, Bouclage;
 
	//initialisation des variables
	errno=0;
	Bouclage = 1;
 
	//déclaration de la taille du socket de communication
	socklen_t SocketCommunicationParamLong;
 
	//création de la socket de connexion
	SocketConnexion = socket(PF_INET, SOCK_STREAM, 0);
	if(SocketConnexion == -1)
	{
		printf("erreur sur socket : %s\n", strerror(errno));
		return(1);
	}
 
	//initialisation de l'adresse de connexion
	bzero(&SocketConnexionParam, sizeof(SocketConnexionParam));
	SocketConnexionParam.sin_family = PF_INET;
	SocketConnexionParam.sin_port = htons(Port);
	SocketConnexionParam.sin_addr.s_addr = inet_addr(AdresseIP);
 
	//liaison entre le nom et la socket de connexion
	if(bind(SocketConnexion, (struct sockaddr *) &SocketConnexionParam, sizeof(SocketConnexionParam)) != 0)
	{
		printf("erreur sur bind : %s\n", strerror(errno));
		close(SocketConnexion);
		return(1);
	}
 
	//mise en écoute de la socket de connexion
	if(listen(SocketConnexion, 10) != 0)
	{
		printf("erreur sur listen : %s\n", strerror(errno));
		close(SocketConnexion);
		return(1);
	}
 
	while(1)
	{
		//accepte les connexions entrantes
		SocketCommunicationParamLong = sizeof(SocketCommunicationParam);
		SocketCommunication = accept(SocketConnexion, (struct sockaddr *) &SocketCommunicationParam, &SocketCommunicationParamLong);
 
		//test si l'accept est bien passé
		if(SocketCommunication != -1)
		{
			//boucle tant que bouclage = 1
			while(Bouclage)
			{
				//lecture du choix
				Choix = LectureChoix(SocketCommunication);
 
				//test si bonne lecture
				if(Choix == -1);
				{
					//erreur de lecture
					printf("erreur sur LectureChoix\n");
					return(1);
				}
 
				//choisi la bonne action suivant choix
				switch(Choix)
				{
					//choix d'arret du serveur
					case ArretServeur :
						close(SocketCommunication);
						close(SocketConnexion);
						return(0);
 
					//choix d'arret de la connection - sortie de la boucle while
					case Fermeture :
						close(SocketCommunication);
						Bouclage = 0;
						break;
 
					//choix d'execution du 1er script
					case Commande1 :
						//lancement d'execution
						if(Execution(SocketCommunication, Commande1) == -1)
						{
							printf("erreur sur Execution\n");
							return(1);
						}
 
						break;
 
					//choix d'execution du 1er script
					case Commande2 :
						//lancement d'execution
						if(Execution(SocketCommunication, Commande2) == -1)
						{
							printf("erreur sur Execution\n");
							return(1);
						}
 
						break;
 
					//choix d'execution du 1er script
					case Commande3 :
						//lancement d'execution
						if(Execution(SocketCommunication, Commande3) == -1)
						{
							printf("erreur sur Execution\n");
							return(1);
						}
 
 
						break;
 
					default :
						//envois du status MauvaiseCommande
						if(EnvoisStatus(SocketCommunication, MauvaiseCommande) == -1)
						{
							//erreur d'écriture
							printf("erreur sur EnvoisStatus\n");
							return(1);
						}
						break;
 
				}//fin choix
			}//fin while(Bouclage)
		}
		else
		{
			//erreur sur accept()
			printf("erreur sur accept : %s\n", strerror(errno));
			close(SocketConnexion);
			return(1);
		}
	}//end while(1)
}

Ca compile déjà mieux. Merci.
Je regarde du coté de eof().

EDIT2 : d'après la FAQ, il faut utiliser feof() et ferror(). J'ai modifié mon code, ca compile. Merci de ton aide.

[publicStaticVoidMain]

N'oublie pas le tag résolu si t'as finalement résolu le probleme.