Discussion :

[GLDavid]

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 :

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
 
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.

@++

[Jack_serious]

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

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[GLDavid]

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 :

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
 
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.

@++

[gege2061]

Salut

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
t = time(NULL);
while(t-start < delay[0]){

ça ne serais pas plutot :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

start = time(NULL);

[joellel]

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
 
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...

[GLDavid]

Salut


ça ne serais pas plutot :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

start = time(NULL);

Salut Gégé

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

@++

[gege2061]

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

Voici une autre manière d'implémenter un timeout : Tuer processus fils

[PRomu@ld]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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) ?