problème de fils

problème de fils - C - Programmation

Marsh Posté le 08-04-2006 à 19:22:48    

Bonjour;
je suis en tarin de réaliser un programme C qui simule un shell. J'ai commancé par creer une procédure "readcmd"
qui lits de l'entrée stnadard une commande tapé par l'utilisateur et la stocke dans une "struct cmdline *l;
Bon mon programme réalise déjà pas mal de chose.
Ce que je voudrais faire maintenant, c'est de lui donner la possibilité d'afficher à l'ecran la terminaiseon d'un processus lancé en tache de fond.
C'est à dire, grace à la fonction "signal", je demande à mon programme d'afficher une phrase qui indique explicitement la terminaison d'un processus fils:  
signal(SIGCHLD,afficher_terminaison);
 
 
 
voila mon programme pour que les choses soient claires:
 

Code :
  1. /*
  2. * Copyright (C) 2002, Simon Nieuviarts
  3. */
  5. #include <signal.h>
  6. #include <sys/stat.h>
  7. #include <fcntl.h>
  8. #include <stdlib.h>
  9. #include <string.h>
  10. #include <sys/types.h>
  11. #include <sys/wait.h>
  12. #include <unistd.h>
  13. #include <stdio.h>
  14. #include "readcmd.h"
  15. #define N 100
  17. int main(){
  19. int desc_sor,desc_ent;//c'est un descripteur du fichier vers lequel on redirige la sortie(ou l'entrée respectivement)
  20. int tuyau[2],length;
  21. pid_t tab_pid[N];//un tableau qui contiendra les pid des processus lancés en background
  22. char* tab_com[N];//un tableau qui contiendra les nom des processus lancés en background
  23. int comp = 0; //un compteur commun pour les deux tableaux précédants
  24. pid_t res,pid;
  25. int* status;
  26. int arg=0; //c'est pour parcourir les arguments d'une commande
  28. void afficher_terminaison() //pour le SIGCHILD
  29. {
  30.      printf("le processus lancé en background est s'est terminé \n" );
  31. }
  35. while (1) {
  36.         struct cmdline *l;
  37.  int i, j;
  39.  printf("shell> " );
  40.  l = readcmd();
  42.  /* If input stream closed, normal termination */
  43.  /*if (!l) {
  44.             printf("exit\n" );
  45.   exit(0);
  46.  }*/
  48.  if (l->err) {
  49.   /* Syntax error, read another command */
  50.   printf("error: \a %s\n", l->err);
  51.   continue;
  52.  }
  54.  if (l->in){
  55.                    printf("in: %s\n", l->in);
  56.                    desc_ent = open(l->in,O_RDONLY);
  57.  }
  58.         if (l->out){
  59.                    printf("out: %s\n", l->out);
  60.                    desc_sor = open(l->out,O_WRONLY|O_CREAT);
  61.         } 
  63.  /* Display each command of the pipe */
  64.  for (i=0; l->seq[i]!=0; i++) {
  65.   char **cmd = l->seq[i];
  66.   printf("seq[%d]: ", i);
  67.   for (j=0; cmd[j]!=0; j++) {
  68.    printf("%s ", cmd[j]);
  69.   }
  70.   printf("\n" );
  71.  }
  72.                
  73.                 if (strcmp(l->seq[0][0] , "exit" ) == 0)
  74.                         {exit(0);}
  75.      
  76.        /*creation d'un pipe au cas ou on a 2 commandes separées par un pipe*/
  77.        if (l->seq[1] != 0)
  78.        {
  79.             pid = fork();
  80.             if (pid > 0) waitpid(pid,NULL,WUNTRACED);
  81.             if (pid == 0 ) //"pid" est le pid du processus à gauche du pipe         
  82.             {
  83.                if(pipe(tuyau)==-1) {
  84.                      printf("erreur a la creation du pipe\n" );
  85.                      exit(2);
  86.                }
  87.                res = fork();
  88.                if(res == 0) { //"res" est le pid du processus à droite du pipe
  89.                     /* je suis le processu à doite */
  90.                     close(tuyau[1]);
  91.                     close(0);
  92.                     dup(tuyau[0]);
  93.                     printf("ici2" );
  94.                     execvp(l->seq[1][0],l-> seq[1]);
  95.                     printf("probleme dans le processus droit\n" );
  96.                     exit(1);
  97.                }
  98.                
  99.                if(res==-1) {
  100.                     printf("erreur a la creation du processus droit\n" );
  101.                     exit(2);
  102.                }
  103.                //processus gauche
  104.                close(tuyau[0]);
  105.                close(1);
  106.                dup(tuyau[1]);
  107.                printf("ici1" );
  108.                execvp(l->seq[0][0],l-> seq[0]);
  109.                printf("probleme dans le processus gauche\n" );
  110.                exit(1);
  111.             }
  112.        }
  113.        else
  114.        { 
  116. /*seq[i] est la ième commande dans la suite des commandes séparés par des pipes,
  117. et seq[i][0] nous donne le nom du processus correspondant à cette commande*/               
  119.                   if (l->seq[0] != 0) {
  120.                   pid = fork();
  121.                   if ((pid == 0) && (l->out != NULL))
  122.                   {
  123.                      close(1);
  124.                      dup2(desc_sor,1);
  125.                   };
  126.                   if ((pid == 0) && (l->in != NULL))
  127.                   {
  128.                      close(0);
  129.                      dup2(desc_ent,0);
  130.                   };
  131.                   if (pid != 0) // Le père
  132.                   {
  133.                       if (strcmp(l->seq[0][0] , "liste_ps" ) == 0)
  134.                       {
  135.                            printf("\n" );
  136.                            printf("pid      " );
  137.                            printf("commande" );
  138.                            printf("\n" );
  139.                            for (i = 0;i<comp; i++)
  140.                            {  printf("%d      ",tab_pid[i]);
  141.                               printf("%s      ",tab_com[i]);
  142.                               printf("\n" );
  144.                            }
  145.                            printf("\n" );
  146.                        }
  147.                        
  148.                   }
  149.                 arg = 0; //réinitialisation de arg     
  150.                 while ( l->seq[0][arg] != 0) { arg++;};//(arg -1) est le nombre d'arguments de la commande (avec le nom de la commande compris)
  151.                 if (strcmp(l->seq[0][arg -1] , "&" ) != 0)
  152.                 {
  153.                   /*on n'attend le fils que si le  processus correspondant n'est
  154.                   pas lancé en background*/
  155.                   waitpid(pid,status,WUNTRACED);
  156.                   if (pid == 0)
  157.                   {
  158.                      execvp(l->seq[0][0],l-> seq[0]);
  159.                      if (strcmp(l->seq[0][0] , "liste_ps" ) != 0)
  160.                      {//liste_ps est une commande que nous avons fabriqueé, donc il ne faut pas la considérer comme incorrecte
  161.                         printf("commande incorrecte \a \n" );
  162.                      } 
  163.                      exit(1);
  165.                   }
  166.                 }
  167.                 else
  168.                 {
  169.                   signal(SIGCHLD,afficher_terminaison);
  170.                   waitpid(pid,status,WNOHANG);
  171.                   l-> seq[0][arg - 1] = NULL; 
  172.                   tab_pid[comp] = pid;
  173.                   length = strlen(l->seq[0][0]) + 1;
  174.                   tab_com[comp] = malloc(length);
  175.                   strcpy(tab_com[comp],l->seq[0][0]);
  176.                   comp++;
  177.                   if (pid == 0)
  178.                   { 
  179.                      execvp(l->seq[0][0],l-> seq[0]);
  180.                      printf("commande incorrecte \a \n" );
  181.                      exit(1);
  182.                   }
  184.                 }
  186.                }
  187. }
  188. }
  189. }


 
 
 
Le problème est que lorsque j'exécute mon programme, la terminaison du premier processus fils se passe bien, mais à partir de la 2ème, il affiche la terminaison avant la fin réelle du processus fils.voila une trace de mon exécution:
 
ensibm:~/SEPC/little_shell>tst
shell> nedit &
seq[0]: nedit &  
shell> le processus lancé en background est s'est terminé  
shell> nedit  &
seq[0]: nedit &  
le processus lancé en background est s'est terminé  
shell>
 
ou bien essayez de l'exécuter vous meme pour mieux voir.
quelqu'un a une explication?
merci d'avance
 
 

Reply

Marsh Posté le 08-04-2006 à 19:22:48   

Reply

Marsh Posté le 08-04-2006 à 19:28:37    

je viens de remarquer qu'il y a beaucoup de fautes d'orthographe dans mon sujet. la honte !
désolé tout le monde...

Reply

Marsh Posté le 08-04-2006 à 23:48:40    

Il me semble que tu envoye le signal avant d'attendre la fin du processus:

Code :
  1. signal(SIGCHLD,afficher_terminaison);
  2.                  waitpid(pid,status,WNOHANG);


en plus tu ne vérifie pas que waitpid retourne un pid valide.

Message cité 1 fois
Reply

Marsh Posté le 09-04-2006 à 00:50:07    

nargy a écrit :

Il me semble que tu envoye le signal avant d'attendre la fin du processus:

Code :
  1. signal(SIGCHLD,afficher_terminaison);
  2.                  waitpid(pid,status,WNOHANG);


en plus tu ne vérifie pas que waitpid retourne un pid valide.


 
 
 
 
mais je n'envoie aucun signal.je dis juste à mon programme qu'il doit effectuer la fonction "afficher_terminaison" quand il reçoit la signal sigchild.
est ce que ça s'exécute correctement chez toi?

Reply

Marsh Posté le 09-04-2006 à 00:58:23    

ha ouais, nawak, pour la peine je sort mon compilo, et....
ça compile pas, entre autre il manque le fichier readcmd.h


Message édité par nargy le 09-04-2006 à 00:58:43
Reply

Marsh Posté le 09-04-2006 à 01:01:40    

il y a aussi la définition de la fonction afficher_terminaison() dans le main() !

Reply

Marsh Posté le 09-04-2006 à 02:25:06    

#
/*
#
* Copyright (C) 2002, Simon Nieuviarts
#
*/
 
 
...

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed