Discussion :

[lornak]

Bonjour,

J'ai créé un programme (qui tourne sous Linux) qui crée 2 processus fils, le premier processus est chargé de réaliser un transfert de fichiers par FTP pendant que le deuxième attend pendant n minutes. Si le transfert FTP ne s'est pas terminé avant le délai imparti, mon programme doit tuer ce processus.

Mon problème est que j'ai l'impression que la commande kill(pid,SIGKILL) ne fait rien !

Voici 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
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
        pid_fils_ftp = fork();  /* creation du premier fils */
        if (pid_fils_ftp==-1)
        {
               /* probleme lors de la creation du fils */
        }
        if (pid_fils_ftp==0)
        {
               /* Transfert FTP */
               exit(0);
        }
        else
        {
            pid_fils_timeout = fork();   /* creation du second fils */
            if (pid_fils_timeout==-1)  
            {
                /* probleme lors de la creation du fils */
            }
            if (pid_fils_timeout==0)    
            {
                sleep(50);        /* attente pendant 50 sec */
                exit(0);
            }
            else
            {
                pid_recup = wait(0);    /* recuperation du pid du premier processus termine */
                if (pid_recup==pid_fils_ftp)
                {
                    /* le transfert FTP s'est termine avant le timeout */
                    kill(pid_fils_timeout, SIGKILL);
                }
                if (pid_recup==pid_fils_timeout)
                {
                    /* le timeout s'est termine avant la fin du transfert FTP */
                    kill(pid_fils_ftp, SIGKILL);
                }
            }
         }

Si vous avez des critiques ou des remarques, n'hésitez pas à m'en faire part. Merci d'avance !

Lornak

[gege2061]

Bonjour,

Code :

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

1
2
3
4
5
 
  if (pid_fils_ftp==-1) 
    /* probleme lors de la creation du fils */
  if (pid_fils_ftp==0)
    [...]

Cela revient à écrire :

Code :

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

1
2
3
4
5
6
7
 
  if (pid_fils_ftp==-1) 
  {
    /* probleme lors de la creation du fils */
    if (pid_fils_ftp==0)
      [...]
  }

Code :

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

pid_recup = wait(0);    /* recuperation du pid du premier processus termine */

Tu ne récupère pas le pid du fils mais tu attend qu'un processus du même groupe que celui ci se termine donc tu peux toujours le tuer par la suite, il est déjà terminé !

[lornak]

Cela revient à écrire :

Code :

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

1
2
3
4
5
6
7
 
  if (pid_fils_ftp==-1) 
  {
    /* probleme lors de la creation du fils */
    if (pid_fils_ftp==0)
      [...]
  }

Oups, je suis allé un peu vite en recopiant le code

Code :

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

 pid_recup=wait(0)

Tu ne récupère pas le pid du fils mais tu attend qu'un processus du même groupe que celui ci se termine donc tu peux toujours le tuer par la suite, il est déjà terminé !

Ok, je pensais récupérer le pid du premier processus terminé. Ainsi si le fils1 était terminé en premier, je tuais le fils2 et vice-versa.

Donc je ne recupère pas le pid, mais n'existe-t-il pas un moyen permettant de connaitre ce pid ?

[gege2061]

Ok, je pensais récupérer le pid du premier processus terminé. Ainsi si le fils1 était terminé en premier, je tuais le fils2 et vice-versa.

Donc je ne recupère pas le pid, mais n'existe-t-il pas un moyen permettant de connaitre ce pid ?

Bon je viens de prendre mon 4 heure, ça va mieux

Ok j'avais pas vu ça comme ça Tu peux faire plus simple :

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_fils_ftp = fork();  /* creation du premier fils */
if (pid_fils_ftp==-1)
{
  /* probleme lors de la creation du fils */
}
else if (pid_fils_ftp==0)
{
  /* Transfert FTP */
  exit(0);
}
else
{
  sleep (50);
  kill (pid_fils_ftp, SIGKILL);
}

Par contre je n'ai pas trouvé de moyen de connaitre l'état d'un processus grâce à son pid (si on tue pid_fils_ftp alors qu'il est déjà terminé, rien ne nous assure qu'un autre processus n'a pas était lancé entre deux avec le même pid ).

Je vais regarder ton code de plus près (avec un compilateur sous la main) parce que normalement ça devrait fonctionner

[Médinoc]

Par contre je n'ai pas trouvé de moyen de connaitre l'état d'un processus grâce à son pid (si on tue pid_fils_ftp alors qu'il est déjà terminé, rien ne nous assure qu'un autre processus n'a pas était lancé entre deux avec le même pid ).

Tu es sûr ? Il me semblait justement qu'un processus ne pouvait être terminé que par son père, donc tant que le père vit, il doit rester quelque chose de ce processus, non ?

[troll mode="velu"]On n'a pas ce genre de problèmes sous Windows...[/troll]

[gege2061]

Tu es sûr ? Il me semblait justement qu'un processus ne pouvait être terminé que par son père, donc tant que le père vit, il doit rester quelque chose de ce processus, non ?

Oui effectivement, le processus devient zombi donc ça devrait marcher

[fearyourself]

Oui effectivement, le processus devient zombi donc ça devrait marcher

Je confirme.

Sinon pour être sûr, faudrait taper dans l'arborescence /proc pour vérifier le ppid du processus. Mais la portabilité du code est alors à revoir car je ne suis pas sûr que toutes les distrib gérent cette arborescence de la même façon.

Jc

[lornak]

J'ai fait appel à un expert qui a conclu que le code fonctionne bien, le problème provient de la façon dont j'effectue le transfert FTP : j'utilise un popen en passant la commande shell "ftp -n .....". Celle-ci crée un processus qui ne se termine que lorsqu'il a consommé toutes les commandes, voilà pourquoi j'avais l'impression que je ne tuais pas le processus.

Afin de tuer le bon processus, la solution consiste à propager le SIGTERM jusqu'au processus créé par le popen.
Pour cela, il faut récupérer le PID par l'intermédiaire du pipe, ouvert en lecture/ecriture en utilisant la commande "ftp -n ... &echo \"PID=$!\";exit $!" , puis lire la ligne PID=n et mettre en place un handler sur SIGTERM de façon à tuer le processus ftp.

Voilà, il ne reste plus qu'à implémenter la solution.

Merci à tous.