problème execv NULL

Version imprimable

problème execv NULL

Bonsoir,

Je veux exécuter des commandes comme ls mais quand je tape des commandes plus longues que deux caractères, les caractères en plus de passent pas dans ma fonction. J'ai trouvé que c'est cette partie du code :

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
 void    loop(t_env *env2, char *buffer)
{
  char  *patate;
  char  **tab1;
  int   i;
 
  i = 0;
  tab1 = init(env2);
  tab1 = my_str_to_wordtab(*my_getenv("PATH", env2->environ), ':');
  i = 0;
  while (tab1[i] != NULL )
    {
      patate = malloc((my_strlen(tab1[i]) + my_strlen(buffer) + 1) * sizeof(char*));
      if (patate == NULL)
        my_putstr_error("errreur amlloc");
      strcat(patate, *tab1);
      del_char(patate, '=');
      del_str(patate, "PATH");
      strcat(patate, "/");
      strcat(patate, buffer);
      i++;
      /* free(patate); */
    }
  /* while (patate[i] != '\0') */
  /*   i++; */
  patate[i] = NULL; /* c'est cette ligne */
  verif(patate);
}

Mais quand je l'enlève, execl m'affiche une erreur.

Merci de votre aide d'avance et bonne soirée.

17/01/2015, 21h54
imperio

Euh... Je ne comprends pas franchement ce que ton code est suppose faire la... Pourquoi ne pas executer la commande avec ce que te retourne la fonction "my_str_to_wordtab" ?
17/01/2015, 22h11
Rémi1995

la fonction my_str_to_wrdtab ne renvoie que le path je le concatène avec la commende à exécuter
17/01/2015, 22h50
diogene
Code:

1 2 3 4 patate = malloc((my_strlen(tab1[i]) + my_strlen(buffer) + 1) * sizeof(char*)); if (patate == NULL) my_putstr_error("errreur amlloc"); strcat(patate, *tab1);
- Tu ne peux pas utiliser strcat() pour copier une chaine : elle sert à concaténer deux chaines. La chaine de départ peut évidemment être vide mais ce doit être une chaine donc terminée par '\0'. Ou tu utilises strcpy() ou avant le strcat() tu places le 0 : *patate = '\0'

- Dans le malloc(), le sizeof(char*) indique que tu veux allouer des pointeurs sur char (auquel cas le type de patate devrait être char**). Tu dois avoir sizeof(char) (ou rien du tout puisqu'il est certain que sizeof(char) ==1)
Code:

1 2 tab1 = init(env2); tab1 = my_str_to_wordtab(*my_getenv("PATH", env2->environ), ':');
Ce code est bizarre : je ne sais pas ce que fait init() ni ce qu'il renvoie, mais dans tous les cas la valeur renvoyée ne sert à rien puisqu'on ne s'en sert pas avant de l'écraser à la ligne suivante.

init sert a malloc tab1 et d'autres variables. et c'est bien strcat car je veux concaténer patate et tab1.

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
 
char    **init(t_env *env2)
{
  char  **tab;
 
  env2->s1 = malloc(4096 * sizeof(char*));
  env2->s1 = *my_getenv("PATH", env2->environ);
  env2->s3 = malloc((env2->len) * sizeof(char**));
  tab = malloc((cpt_word(env2->s1, ':') + 1) * sizeof(*tab));
  return (tab);
}
 
void    loop(t_env *env2, char *buffer)
{
  char  *patate;
  char  **tab1;
  int   i;
 
  i = 0;
  tab1 = init(env2);
  tab1 = my_str_to_wordtab(*my_getenv("PATH", env2->environ), ':');
  i = 0;
  while (tab1[i] != NULL )
    {
      patate = malloc((my_strlen(tab1[i]) + my_strlen(buffer) + 1) * sizeof(char*));
      if (patate == NULL)
        my_putstr_error("errreur amlloc");
      patate[0] = '\0';
      strcat(patate, *tab1);
      del_char(patate, '=');
      del_str(patate, "PATH");
      strcat(patate, "/");
      strcat(patate, buffer);
      i++;
      /* free(patate); */
    }
  /* while (patate[i] != '\0') */
  /*   i++; */
  patate[i] = '\0';
  verif(patate);
}

17/01/2015, 23h28
imperio
Citation:

Envoyé par Rémi1995

la fonction my_str_to_wrdtab ne renvoie que le path je le concatène avec la commende à exécuter

Pour autant que je sache, ce n'est pas du tout ce que cette fonction est censée faire ! Prenons un exemple :
Code:

1 2 3 4 char *str = "je/suis/un/test/"; char **words = my_str_to_wordtab(str, '/'); // contenu de words : ["je", "suis", "un", "test"]
Donc encore une fois, je ne comprends pas ce que tu veux faire...
17/01/2015, 23h37
Rémi1995

e récupère la path entière ce qui me donne:
/bin:/sbin:/usr/bin:/usr/sbin:/usr/heimdal/bin:/usr/heimdal/sbin:/home/boivin_r/bin

ma fonction my_str_to_wordtab me donne

/bin
/sbin
/usr/bin
/usr/sbin
/usr/heimdal/bin
/usr/heimdal/sbin
/home/boivin_r/bin

ensuite je la concatène avec buffer et je stockes les deux dans patate.
18/01/2015, 11h20
diogene
Citation:

init sert a malloc tab1 et d'autres variables...

Je veux bien, mais la ligne qui suit init() détruit l'adresse renvoyée par init().
Code:

1 2 tab1 = init(env2); tab1 = my_str_to_wordtab(*my_getenv("PATH", env2->environ), ':');
Citation:

... c'est bien strcat car je veux concaténer patate et tab1.

J'ai bien compris, mais le premier appel à strcat() veut être en fait une copie donc il fallait initialiser patate. D'ailleurs, tu l'as rajouté dans le code.

- Ma remarque sur le sizeof (char*) est toujours valable.
18/01/2015, 11h57
Rémi1995

donc init ne sert à rien ok merci je vais corriger ça. pour le sizeof char* c'est pour patate ?
18/01/2015, 12h05
diogene

init() ne sert pas à rien puisqu'il initialise env2. C'est uniquement un problème d'allocation de tabl : Qui la fait, init() ou my_str_to_wordtab() ? Il faut choisir.

Pour le sizeof, c'est ici :patate = malloc((my_strlen(tab1[i]) + my_strlen(buffer) + 1) * sizeof(char*)); .

j'ai fais comme ça

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
 
void    loop(t_env *env2, char *buffer)
{
  char  *patate;
  char  **tab1;
  int   i;
 
  i = 0;
  tab1 = init(env2);
  i = 0;
  while (tab1[i] != NULL )
    {
      patate = malloc((my_strlen(tab1[i]) + my_strlen(buffer) + 1) * sizeof(char));
      if (patate == NULL)
        my_putstr_error("errreur amlloc");
      patate[0] = '\0';
      strcat(patate, *tab1);
      del_char(patate, '=');
      del_str(patate, "PATH");
      strcat(patate, "/");
      strcat(patate, buffer);
      i++;
      /* free(patate); */
    }
  /* while (patate[i] != '\0') */
  /*   i++; */
  patate[i] = '\0';
  verif(patate);
}
 
void    prompt(t_env *env2)
{
  char  *buffer;
 
  buffer = malloc(4096 * sizeof(char*));
  if (buffer == NULL)
    my_putstr_error("errreur amlloc");
  else
    {
      my_putstr("$-->");
      memset(&buffer[0], 0, 4096);
      if (read(0, buffer, 4095) <= 0)
        {
          my_putstr("[ERROR]");
        }
      else
        {
          loop(env2, buffer);
        }
    }
}
 
char    **init(t_env *env2)
{
  char  **tab;
 
  env2->s1 = malloc(4096 * sizeof(char*));
  env2->s1 = *my_getenv("PATH", env2->environ);
  env2->s3 = malloc((env2->len) * sizeof(char**));
  tab = malloc((cpt_word(env2->s1, ':') + 1) * sizeof(*tab));
  tab = my_str_to_wordtab(*my_getenv("PATH", env2->environ), ':');
  return (tab);
}

18/01/2015, 13h28
diogene
Dans init(), même défaut que tu ne sembles pas comprendre :
Code:

1 2 tab = malloc((cpt_word(env2->s1, ':') + 1) * sizeof(*tab)); tab = my_str_to_wordtab(*my_getenv("PATH", env2->environ), ':');
On ne peut avoir ce code : la première ligne fait une allocation et la deuxième détruit aussitôt l'adresse de cette allocation. On ne peut avoir que l'une de ces deux lignes. La question est toujours : Qui fait l'allocation, my_str_to_wordtab() ou pas ?

Dans loop() :
A chaque tour de la boucle, tu alloues une nouvelle zone mémoire dont tu mets toujours l'adresse dans une même variable patate. Mais alors, que sont devenues les adresses des zones mémoire allouées précédemment ? Réponse : perdues.

my_str_to_wordtab retourne la chaine de caractère avec la path découpé mais elle ne malloc pas tab. J'ai fais comme ça pour la fonction loop

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
 
void    loop(t_env *env2, char *buffer)
{
  char  *patate;
  char  **tab1;
  int   i;
 
  i = 0;
  tab1 = init(env2);
  i = 0;
  patate = malloc((my_strlen(tab1[i]) + my_strlen(buffer) + 1) * sizeof(char));
  if (patate == NULL)
    my_putstr_error("errreur amlloc");
  while (tab1[i] != NULL )
    {
      patate[0] = '\0';
      strcat(patate, *tab1);
      del_char(patate, '=');
      del_str(patate, "PATH");
      strcat(patate, "/");
      strcat(patate, buffer);
      i++;
    }
  patate[i] = '\0';
  verif(patate);
}

18/01/2015, 17h17
Rémi1995

j'ai résolu le problème mais maintenant je n'arrive pas à changer le morceau de path
par exemple :
/bin/ls OK // on execute
/bin/clear error //on continue
/sbin ...
18/01/2015, 17h57
diogene
Code:

1 2 i = 0; patate = malloc((my_strlen(tab1[i]) + my_strlen(buffer) + 1) * sizeof(char));
Ceci n'alloue la quantité de mémoire correcte que pour le traitement de tab1[0]
Code:

1 2 patate[0] = '\0'; strcat(patate, *tab1);
Ceci vide la chaine patate au début de chaque tour de la boucle puis remet dans patate toujours la même chose, la chaine pointée par tab1[0]
Bref, à chaque tour de boucle, on refait la même chose
18/01/2015, 18h04
Rémi1995

si ce n'ai pas comme ça qu'on fait c'est comment ? pour la path c'est réglé mais comment faire pour qu'ensuite je réaffiche my_putstr("$-->"); ?
19/01/2015, 10h17
Rémi1995

J'ai trouvé pour afficher my_putstr("$-->")
mais j'ai un petit soucis je voudrai utiliser des commandes avec paramètres comme rm mais quand je fais rm *.o il m'affiche error 2. comment faire pour que je puisse lancer des commandes avec paramètres.
19/01/2015, 13h56
imperio

Il faut faire appel à une fonction pour transformer ton *.o en tous les fichiers qui matchent cette expression. Fort heureusement, une fonction C le fait.

OK merci pour la réponse mais maintenant j'ai un autre problème. quand je veux lancer clear il me dit : TERM environment variable not set. j'ai essyer de lancer reset -s xterm mais il m'affiche la même chose. voilà comment j'ai fais

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
 
int     execute(t_env *env2)
{
  int   statut;
  int   i;
  pid_t pid;
  char  **argv1;
 
  i = 0;
  env2->arg = my_str_to_wordtab(env2->commande, ' ');
  argv1 = malloc((my_strlen(env2->arg[0]) + 1) * sizeof(char**));
  while (env2->arg[i])
    {
      argv1[i] = malloc((my_strlen(env2->arg[0]) + 1) * sizeof(char**));
      i++;
    }
  if (argv1[0] == NULL)
    my_putstr_error("Error argv1 is NULL\n");
  if ((pid = fork()) == -1)
    my_putstr_error("[ERROR]:fork problem\n");
  if (pid == 0)
    {
      if (execve(env2->arg[0], env2->arg , NULL) == -1)
        {
          return (-1);
          exit(1);
        }
    }
  else
    wait(&statut);
  return 0;
}

c'est bon j'avais juste oublier de passer env en paramètre.

je n'arrive pas a passer le chemin a la commande chdir.

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
 
void    prompt(t_env *env2)
{
  char  *cd;
  char  *cd1;
  char  *tmp;
  int   i;
 
  i = 0;
  env2->buffer = malloc(4096 * sizeof(char*));
  cd = malloc(my_strlen(env2->buffer) * sizeof(char*));
  cd1 = malloc(my_strlen(env2->buffer) * sizeof(char*));
  tmp = malloc(my_strlen(env2->buffer) * sizeof(char*));
  if (cd == NULL || cd1 == NULL || tmp == NULL)
    my_putstr_error("Error malloc");
  if (env2->buffer == NULL)
    my_putstr_error("errreur amlloc");
  else
    {
      my_putstr("$-->");
      memset(&env2->buffer[0], 0, 4096);
      if (read(0, env2->buffer, 4095) <= 0)
          my_putstr("[ERROR]");
      else
        {
          i = 0;
          my_strncpy(cd,env2->buffer,2);
          my_strcpy(cd1, env2->buffer);
          del_str(cd1, cd);
          while (cd1[i] != '\n')
            {
              tmp[i] = cd1[i];
              i++;
            }
          if (tmp[i] == '\n')
            printf("ee\n");
          if (strncmp(cd, "cd\n", 2) == 0)
            {
              printf("%s\n", tmp);
              if (chdir(tmp) == -1)
                {
                  perror("error\n");
                  /* exit(1); */
                }
              else
                printf("OK\n");
            }
          else
            loop(env2);
        }
    }
}

Tu n'es pas assez attentif.
Code:

1 2 3 4 env2->buffer = malloc(4096 * sizeof(char*)); cd = malloc(my_strlen(env2->buffer) * sizeof(char*)); cd1 = malloc(my_strlen(env2->buffer) * sizeof(char*)); tmp = malloc(my_strlen(env2->buffer) * sizeof(char*));
Les 3 derniers malloc() sont faux pour deux raisons :

- cd, cd1 et tmp (et peut être env2->buffer) sont des char* alors il n'y a pas de raison d'avoir sizeof(char*), mais on devrait avoir sizeof(char) (qui d'ailleurs vaut 1 par définition)

- surtout, my_strlen(env2->buffer) va retourner n'importe quoi puisque la mémoire pointée par env2->buffer contient n'importe quoi : malloc() alloue de la mémoire mais la donne dans un état quelconque

23/01/2015, 21h29
Christophe
sizeof(char) va retourner la taille d'un char : soit 1 caractère donc 1 octet.
sizeof(char*) va retourner la taille d'un pointeur de char, donc une adresse mémoire. Si en 32 bits : 4 octets si 64 bits : 8 octets.
Code:

1 2 malloc(my_strlen(env2->buffer) * sizeof(char*));
a pour résultat de réserver le contenu de env2->buffer multiplié par 4. (en supposant que tu es en 32 bits)
Que contient env2->buffer ? une adresse mémoire. (comme signalé par Diogène). Tu réserves donc x*4 (x au sens mathématique) dont en plus je ne vois pas la libération.

Ton but est de préparer une structure arge pour execv ? ( ou peut être argv)

Que transfert tu ?
23/01/2015, 22h47
sambia39

Bonsoir
Très énigmatique ton programme, je ne sais pas vraiment quel est le but final mais, vu les différents bouts de codes je comprends une exécution de programme à travers ton programme ? et je me demande si tu t'es réellement informé sur les différentes fonctions système que tu utilises.
pour ma part, je te conseille ce tutoriel en complément http://mtodorovic.developpez.com/lin...e/?page=page_3
à bientôt

Fuseau horaire GMT +2. Il est actuellement 08h41.