[C system] timeout sur un processus

Version imprimable

Bonjour

Voulant me remettre au C, je me suis fixé ce petit projet : créer une commande timeout qui prend un temps en secondes et une commande shell. Si au bout de ce temps t la commande n'est pas terminée, elle est tuée.
A priori, tout va bien dans mon code, aucune erreur à la compilation avec gcc v4.0.3 sous Ubuntu 6.06 x86. Mais à L'exécution, malgré le temps donné, la commande ne se fait pas tuer.
Voici la partie de mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 pid = fork(); if(pid<0){ fprintf(stderr, "Fork error : %d\n", pid); exit(pid); } if(pid==0){ t = time(NULL); while(t-start < delay[0]){ execvp(command[0], command); t = time(NULL); } kill(pid, SIGKILL); fprintf(stdout, "Timeout.\n"); exit(EXIT_SUCCESS); } else{ retour = waitpid(pid, &status, 0); if(retour < 0){ exit(EXIT_SUCCESS); } }
Peut être que l'utilisation du while n'était pas une bonne idée ?

Merci d'avance de vos conseils.

@++

Primo, j'espere pour toi que start est initialise :mrgreen:.

Deuxiemement, le decoupage n'est pas bon.

La fonction execvp(3) lance l'execution d'une autre commande, ce qui fait que tout le code present par la suite ne sera jamais execute.

A mon avis ca devrait plus ressembler a ca:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 pid = fork(); if (pid < 0) { /* traitement de l'erreur */ } else if (pid == 0) { /* execvp ici */ } else { /* attente de la fin du timeout ou de la mort du processus */ /* kill du processus si timeout ecoule */ }

Hello Jack

Merci de ta réponse.
Ok, je vais tâcher de modifier mon code en ce sens. Ce qui me semble bizarre, c'est que j'avais placé initialement ma boucle while dans le traitement du processus du père.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 pid = fork(); if(pid<0){ fprintf(stderr, "Fork error : %d\n", pid); exit(pid); } if(pid==0){ execvp(command[0], command); } else{ t = time(NULL); while(t-start < delay[0]){ retour = waitpid(pid, &status, 0); if(retour < 0){ exit(EXIT_SUCCESS); } t = time(NULL); } kill(pid, SIGKILL); fprintf(stdout, "Timeout.\n"); exit(EXIT_SUCCESS); }
Or, même avec ce code, le fils ne mourrait pas lorsque le timeout est atteint. Mais peut être me suis-je encore planté (sans doute !). Une précision, le processus fils s'exécute bien.

Merci encore de votre aide.

@++

03/07/2006, 08h47
gege2061
Salut ;)
Citation:
Envoyé par GLDavid

Code:

1 2 t = time(NULL); while(t-start < delay[0]){
ça ne serais pas plutot :

Code:

start = time(NULL);

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
pid = fork();
	if(pid<0){
		fprintf(stderr, "Fork error : %d\n", pid);
		exit(pid);
	}
	if(pid==0){
		execvp(command[0], command);
	}
	else{
                          t = time(NULL);
		while(t-start < delay[0]){
		   retour = waitpid(pid, &status, 0);
		   if(retour < 0){
			exit(EXIT_SUCCESS);
		   }	
		  t = time(NULL);
	            }    
	          kill(pid, SIGKILL);
		fprintf(stdout, "Timeout.\n");
		exit(EXIT_SUCCESS);
	}

Je ne comprends pas bien ton code: si tu entres dans le while, tu n'en sortiras pas tant que le fils ne s'est pas terminé (waitpid() attend la fin du fils). Donc tu n'exécuteras jamais ton kill()....
Ou alors, je suis mal réveillée...

03/07/2006, 08h53
GLDavid

Citation:

Envoyé par gege2061

Salut ;)

ça ne serais pas plutot :

Code:

start = time(NULL);

Salut Gégé

Le start est initialisé avant, donc pas de souci à ce niveau là.

@++
03/07/2006, 08h59
gege2061

Citation:

Envoyé par GLDavid

Le start est initialisé avant, donc pas de souci à ce niveau là.

Ok, j'espère pour toi que le temps entre les deux n'est pas supérieur au timeout :lol:

Voici une autre manière d'implémenter un timeout : Tuer processus fils
03/07/2006, 09h01
PRomu@ld

Code:

waitpid(pid, &status, 0);

Le problème vient de là, tu attends la fin du fils histoire de voir si il ne se termine pas avant le timer, c'est bien mais cet appel est bloquant donc ton programme ne fera qu'attendre la fin du fils et le timer n'existera pas.

Tu as une solution, qui consiste à rendre l'appel non bloquant (en mettant le troisième paramètre à WNOHANG)

Autre chose, pourquoi ne passes tu pas par un signal (SIGALRM) ?