Discussion :

[Raikyn]

Salut,

J'ai un petit soucis sur mon programme, en fait je fais un appel à fork() pour donner une tâche à faire à un fils, le detruire, re-donner une tâche à un autre fils ainsi de suite (il s'agit en fait d'un "mini shell" qu'on doit programmer).

Donc mon code ressemble à (je vous épargne les détails):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int main(){
   while(1){
   ...
   //traitement des données entrées par l'utilisateur, stockage des commandes dans un   tableau
   ...
   int pid = fork();
   if(pid ==0){
           //on execute les commandes via la fonction execvp(...)
            break;
    }
   wait() //celui est nécessaire mais je sais pas si je dois le placer ici (c'est pour permettre au père d'attendre avant de refaire un tour de boucle)
   }
}

Mon problème c'est que ça rentre pas dans mon "if" ligne 7. J'ai fait des printf du pid et ça me met autre chose que 0, alors que normalement le fils doit avoir 0 en pid non ?

Merci d'avance pour votre aide.

[Bktero]

Le PID du fils est renvoyé au père, est-ce que le PID que tu affiches ne serait pas celui-là ?

http://man.developpez.com/man2/fork.2.php

VALEUR RENVOYÉE

En cas de succès, le PID du fils est renvoyé au processus parent, et 0 est renvoyé au processus fils. En cas d'échec -1 est renvoyé dans le contexte du parent, aucun processus fils n'est créé, et errno contient le code d'erreur.

[imperio]

J'avais pour habitude de mettre l'appel à wait dans un else mais je ne sais si ça changerait quelque chose étant donné que tu appelles execvp qui termine ton processus à la fin de son exécution. Par contre si l'appel à execvp a échoué, il faudra que tu termines le processus toi-même si je me souviens bien (en appelant exit je présume...).

Il serait bien aussi que tu vérifies si la fonction fork n'a pas échoué (comme l'a cité Bktero, elle renvoie -1 dans ce cas-là).

[moins1]

T'as des constantes définies pour le retour de fork(), ce serait peut être pas une mauvaise idée de vérifier quelle valeur errno a.

man fork genre.

Peut être avec perror().

[Raikyn]

Ok j'ai modifié un peu mon truc, le fork() me retourne bien 0, mais mon execvp ne s'exécute pas ... Pourtant je suis sûr de que ça doit bien faire qqchose (je l'ai testé dans un fichier à part).

Code :

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

1
2
3
4
5
6
7
8
9
 int pid = fork();
    printf("  %d   ",pid);
    if(pid == 0){ 
      execvp(mots[0],mots);
 
      }
    else{
      wait();
    }

[imperio]

Affiche nous le contenu de ta variable mots et vérifie le retour de execvp car je pense que tu ne lui passes pas les arguments attendus.

[Raikyn]

L'erreur renvoyée est :

Code :

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

1
2
3
ls: invalid option -- '
'
Try `ls --help' for more information.

Alors dans mots (tableau de char) :

Code :

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

1
2
3
for(y=0;y<i;y++){
      printf("%s ",mots[y]);
    }

Et par exemple si je rentre "ls -l", ça affiche bien ls dans mots[0] et -l dans mots[1].


Et dans mon fichier test j'ai recrée le execvp tel que :

Code :

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

1
2
3
4
5
6
7
8
int main(){
  char ** arg = malloc(2);
  arg[0] = "ls";
  arg[1] = "-l";
  execvp(arg[0],arg);
 
  return 0;
}

Et cela marche très bien.

[Metalman]

execvp : NULL terminated !

arg = malloc(3);

arg[2] = NULL !

Lire man pages !

man 3 execvp

EDIT : désolé pour la rapidité... au moins on confirme le problème ainsi !

[ram-0000]

Et dans mon fichier test j'ai recrée le execvp tel que :

C'est pas bon, ca c'est bon :

Code :

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

1
2
3
4
5
6
7
8
9
int main(){
  char ** arg = malloc(3);
  arg[0] = "ls";
  arg[1] = "-l";
  arg[2] = NULL;
  execvp(arg[0],arg);

  return 0;
}

Edit : grillé par Metalman

[Raikyn]

Ok j'ai mis le NULL en dernier dans mon tableau, j'ai compilé, j'ai testé en entrant "ls -l" mais ça ne marche toujours pas, j'ai toujours la même erreur. Par contre j'ai essayé un "mkdir dos" et là ça marche (enfin presque : le dossier est créer mais le nom est "dos?").

[Metalman]

Tu rentres comment ton "ls -l" ?
Si c'est par la ligne de commande, tu devrais peut être copier ton argv depuis la case [1]...

Si c'est en dur avec le code de ram_0000... je ne vois pas trop d'où le problème peut venir....

[imperio]

Ok j'ai mis le NULL en dernier dans mon tableau, j'ai compilé, j'ai testé en entrant "ls -l" mais ça ne marche toujours pas, j'ai toujours la même erreur. Par contre j'ai essayé un "mkdir dos" et là ça marche (enfin presque : le dossier est créer mais le nom est "dos?").

Toi tu lis dans la mémoire ou alors t'initialises mal ta chaîne de caractère !

[Raikyn]

Désolé je ne comprend pas ^^

Je fais mon tableau mots tel que :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 char **mots = malloc(i+1);
char *pointeur = NULL;
 
    pointeur = strtok(c2," ");
    int k = 1;
    mots[0] = pointeur;
    while(pointeur != NULL){
      pointeur = strtok(NULL," ");
      mots[k] = pointeur;
      k++;
    }
    /* ------------------- */
    mots[i] = NULL;

Les chaines de caratères me semblent bien initialisées. Ca me rend dingue ça ^^

[Metalman]

Attention attention attention !

Strtok ne duplique pas la chaine !
Il transforme les tokens en NULL et refile les pointeurs sur chaque debut de mot !
Tu dois récupérer ton pointeur et faire un strdup propre pour être "tranquille" !
(strdup n'est pas dans ANSI au passage... mais ses sources sont dispos un peu partout, ou dispo si tu retires le flag -ansi)

Peux-tu nous montrer ton i aussi ?
EDIT : et nous montrer c2 aussi, tant qu'à faire...

[Raikyn]

Alors voilà ^^ :

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
39
40
41
42
43
44
45
46
47
48
49
50
int main(){
  char * c = malloc(30);
  char *pointeur = NULL;
  char * c2 = malloc(30);
  int i = 0;
 
  while(1){
    printf("$");
    fgets(c,30,stdin);
 
 
    /* ---PARSING--- */
    strcpy(c2,c);
    int i = 0;
    pointeur = strtok(c," ");
    while(pointeur != NULL){
      pointeur = strtok(NULL," ");
      i++;
    }
    char **mots = malloc(i+1);
 
    pointeur = strtok(c2," ");
    int k = 1;
    mots[0] = pointeur;
    while(pointeur != NULL){
      pointeur = strtok(NULL," ");
      mots[k] = pointeur;
      k++;
    }
    /* ------------------- */
    mots[i] = NULL;
 
 
    int pid = fork();
    //printf("  %d   ",pid);
    if(pid == 0){
 
      execvp(mots[0],mots);
 
      }
    else{
      wait();
    }
 
  }
 
 
  return 0;
 
}

Je fais un premier tour avec strtok pour compter le nombre de mot pour pouvoir bien allouer la taille de mémoire pour **mots.
Ensuite un deuxième tour pour allouer les mots dans **mots.
Enfin mon if avec le fork() pour faire le execvp.


C'est peut-être une implémentation pas très rigoureuse, mais j'ai bien ce que je souhaite dans **mots.
Après comme dit ci-dessus, peut-être qu'il y a un problème quand j'accède à la mémoire, mais pourquoi le mkdir marche et pas le "ls -l" ?

[ternel]

parce que ton mkdir ne marche pas, comme tu le vois.
il est possible que ce soit ls qui échoue, parce qu'il ne comprendrait pas l'option "-l?"

[Metalman]

Il ne faut pas faire 2 strtoks, malheureux !
Tu dois d'abord compter le nombre de mots, puis faire le strtok !

un petit :

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
 
int nb_word(char *str, char c)
{
 int i = 0;
 int res = 1;
 
 while ((str[i] != NULL) && (str[i] == c))
  i++;
 
 while (str[i] != NULL)
 {
  if (str[i] == c)
  {
    res++;
    while ((str[i] != NULL) && (str[i] == c))
      i++;
  }
  else
    i++;
 }
  return (res);
}

Te fera compter (je suis trop gentil)

EDIT : et RES doit démarrer à 1 plutôt....
2e EDIT et si tu démarres avec du vide...

[Raikyn]

Ok merci, c'est vrai que c'est plus agréable comme ça.

Ce que j'ai fais c'est que j'ai copié la chaine de caractère, c'est-à-dire que je fais un strtok sur la 1ère chaine et un 2eme strtok sur la copie, donc ça marche aussi non ? Vu que mon tableau mots est rempli avec les bons caractères.

parce que ton mkdir ne marche pas, comme tu le vois.
il est possible que ce soit ls qui échoue, parce qu'il ne comprendrait pas l'option "-l?"

Même le ls tout seul ne marche pas, le mkdir/rm marche même quand je fais un "emacs hello.c" ><

[Metalman]

Logiquement oui ça doit marcher...
N'oublie pas le memset à 0 pour tes 30 cases !

[Raikyn]

J'ai plutôt l'impression que le ls s'affiche mais pas dans mon terminal ... Je ne dois pas rediriger la sortie pour que ça s'affiche dans mon mini shell ?