(execve et EFAULT) Shell perso ;)

Version imprimable

22/05/2009, 19h46
leoz2007

(execve et EFAULT) Shell perso ;)

Bonjour,
je suis actuellement entrain de coder un shell mais je rencontre un problème assez délicat.

mon shell exécute toutes les binaires (ls,...)
j'ai coder un parseur pour gérer le cas ou dans ma chaine de caractère comporte une etoile.

ex:
>> ls *.c
mon parseur recupere le "mot" *.c et le remplace par tout les .c du fichier courant (jusque la pas de probleme ;), je print la ligne est elle est correct.)
cependant execve m'envoi l'erreur ERRNO 14 soit EFAULT.

cet erreur correspond a
EFAULT
L'argument fichier pointe en dehors de l'espace d'adressage accessible.

apres plusieur teste je me rend compte que mon shell n'execute :
ni "ls *.c"
ni ls ma*.*
mais qui gere bien "ma*.c"

Quelqu'un aurait-il une idee ?

Cordialement
Leoz
22/05/2009, 20h48
kamouminator

salut !
pour pouvoir t'aider il faudrait qu'on voit ton code. on peut pas deviner tes erreurs ^^

et voila voili voilou les bouts de code concerner ^^

Merci :)

Code:

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
 
char    *parse_star(char *str)
{
  char  **tabl;
  char  **retab;
  char  *str2;
  int   i;
  int   e;
 
  e = 0;
  i = 0;
  str = space(str);
  tabl = str_to_wordtab(str, " ");
  retab = xmalloc(sizeof(*retab) * strlen(str));
  while (tabl[i] != NULL)
    {
      retab[e] = star(tabl[i]);
      e++;
      i++;
    }
  retab[e] = NULL;
  str2 = tab_to_str(' ', retab);
  return (str2);
}
 
 
-------------------------------------------------------------
 
static void     exec_wait(t_info *info)
{
  int           statut;
  int           i;
 
  i = 0;
  if (info->nb_wait > 0)
    while (i++ < info->nb_wait)
      wait(&statut);
  info->nb_wait = 0;
}
 
int             execute(t_mysh *mysh)
{
  t_pipe        *lpipe;
  t_cmd         *lcmd;
  t_cmp         cmp;
 
  cmp.i = 0;
  lcmd = mysh->icmd->begin;
  while (cmp.i <  mysh->icmd->len_list)
    {
      cmp.j = 0;
      mysh->info->flag = FALSE;
      lpipe = lcmd->begin_pipe;
      mysh->info->std = lcmd->len_list_pipe;
      while ((cmp.j < lcmd->len_list_pipe) || (mysh->info->std > 0))
        {
          check_cmd(epur_str(lpipe->table_pipe), mysh);
          mysh->info->flag = TRUE;
          lpipe = lpipe->next;
          mysh->info->std--;
          cmp.j++;
        }
      exec_wait(mysh->info);
      lcmd = lcmd->next;
      cmp.i++;
    }
  return (TRUE);
}
-------------------------------------------------------------------
#include <string.h>
#include <unistd.h>
#include <stdlib.h>
#include <errno.h>
 
# define        MSG_ERROR_EXECVE        "2: execve c'est mal terminer!\n"
 
int                     xexecve(char *bin_path, char **bin_cmd)
{
  extern        char    **environ;
  int                   fd;
 
  fd = 0;
  fd = execve(bin_path, bin_cmd, environ);
  if (fd == (-1))
    {
      printf("errno => %i\n", errno);
      write(2, MSG_ERROR_EXECVE, strlen(MSG_ERROR_EXECVE));
      exit(0);
    }
  return (fd);
}

22/05/2009, 21h09
kamouminator

pourrais tu indenter ton code ? et me dire ou est passée la fonction main ! ^^
je suppose que c'est dans le main que tu appel xexecve ?

en fait c'est un shell complet recoder par nos soins (etudiant d'une ecole d'info), je pensse qu'il serait plus judicieux de te fournir un .rar plutot que fichier par fichier ^^

allez un petit main juste pour te montrer ^^
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 int main() { t_mysh *mysh; char area[BUF_SIZE + 1]; char *term; mysh = xmalloc(sizeof(*mysh)); malloc_all(mysh); if ((term = check_error()) != NULL) init_cap(mysh->cap, area); signal(SIGWINCH, show_win_size); path_search(mysh->env); add_old_story(mysh->story, ARG_STORY); wait_cmd(mysh, mysh->info); xtputs(mysh->cap->maudible_bell, 1, my_outc); free_all(mysh); return (EXIT_SUCCESS); }
Ps: je peut pas indenter dans les balises quote...

22/05/2009, 21h17
kamouminator

Citation:

Envoyé par leoz2007

Ps: je peut pas indenter dans les balises quote...

alors utilse la balise CODE ^^ c'est le #
22/05/2009, 21h24
kamouminator

ca ne me dis toujours pas où se fais l'appel à execve...
22/05/2009, 21h27
kamouminator

le top serait que tu mette un breakpoint sur execve et que tu me donne le backtrace.
ce que je veux savoir en fait, ce sont les arguments que tu passe à execve

tu veu mon rar ?

je l'apelle dans wait_cmd
mais comme je te dis mon rar fait 12mo.

Code:

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
 
#include <stdio.h>
#include <stdlib.h>
#include "mysh.h"
#include "cmd.h"
 
 
int     wait_cmd(t_mysh *mysh, t_info *info)
{
  while (info->continu == TRUE)
    {
      mputstr(info->prompt);
      init_index_struct_cmd(mysh->icmd);
      mysh->story->index = 0;
      init_index_struct_list(mysh->story);
      mysh->cmd = get_line(mysh);
      mysh->cmd = parse_star(mysh->cmd);
      printf("mysh->cmd (star) => %s\n", mysh->cmd);
      mysh->cmd = parse_exclamation(mysh->cmd, mysh);
      add_list_story(mysh->story, mysh->data->cmd);
      printf("mysh->cmd :%s\n", mysh->cmd);
      if (analyse(mysh, mysh->cmd) == TRUE)
        execute(mysh);
      free_list_cmd(mysh->icmd);
    }
  mputstr("Good Bye!\n");
  return (EXIT_SUCCESS);
}

22/05/2009, 21h35
kamouminator

Citation:

Envoyé par leoz2007

tu veu mon rar ?

si le code compile, oui.
22/05/2009, 21h37
kamouminator

et vire les binaires de ton archive...
22/05/2009, 21h44
leoz2007

mon acces ftp est impossible depuis l'ecole une idee pour t'envoyer sa, en piece jointes par email si tu veu bien me communiquer ton mail en pv ;)

sinon je regarde du coter de google code search mais toutes ;a promo check les files uploader ;)
22/05/2009, 21h47
kamouminator

essaye ca:
http://dl.free.fr/
22/05/2009, 21h53
leoz2007

http://dl.free.fr/iBo1echoe

regarde tes mp :)
Merci ;)
22/05/2009, 23h26
kamouminator
il semblerai qu'une fois que * ai été interpreté ton programme plante dans tab_to_str:
Code:

1 2 str = xmalloc(sizeof(char) * (tablen(tabb) + 1)); memset(str, '\0', strlen(str));
str viens d'etre alloué, tu ne peut pas recuperer sa taille avec strlen
replace

Code:

strlen(str)

par:

Code:

tablen(tabb) + 1

ca devrait regler le probleme...s'il n'y en a pas d'autres...j'ai pas testé, mais en tout cas, c'est ce qui le faisait planter
22/05/2009, 23h43
leoz2007

Malheureusement non, il doit y avoir plusieurs problème :/

il suffit de passer les argument ls -l -a -G -d (par exemple)
et non pas ls -laGd ;)

ls -l -a -G -d me pete une erreur errno 14...

Merci pour ton message deja ;)
23/05/2009, 00h10
kamouminator

il semblerai que ca vienne de:

Code:

check_cmd(epur_str(lpipe->table_pipe), mysh);

dans execute.c

il est tard je regarderai ce weekend.
bon courage !
23/05/2009, 00h12
leoz2007

merci c'est gentil ;)
moi je pars pour 72h de code intensse ;)

Bonne nuit
23/05/2009, 19h21
kamouminator

il semblerai aussi qu'il y ai un problème de saisie:
si tu tape:

Code:

ls -l -a -G -d

et que tu essaie d'effacer, tu n'y arrivera pas !
sinon je regarde pour le second bug...