problème execv NULL

**Rémi1995** · 17/01/2015, 19h58

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 :

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
 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.

**imperio** · 17/01/2015, 20h54

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" ?

**Rémi1995** · 17/01/2015, 21h11

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

**diogene** · 17/01/2015, 21h50

Code :

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

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 :

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

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.

**Rémi1995** · 17/01/2015, 21h54

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

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
 
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);
}

**imperio** · 17/01/2015, 22h28

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 :

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

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...

**Rémi1995** · 17/01/2015, 22h37

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.

**diogene** · 18/01/2015, 10h20

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 :

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

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

... 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.

**Rémi1995** · 18/01/2015, 10h57

donc init ne sert à rien ok merci je vais corriger ça. pour le sizeof char* c'est pour patate ?

**diogene** · 18/01/2015, 11h05

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*)); .

**Rémi1995** · 18/01/2015, 11h35

j'ai fais comme ça

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
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);
}

**diogene** · 18/01/2015, 12h28

Dans init(), même défaut que tu ne sembles pas comprendre :

Code :

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

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.

**Rémi1995** · 18/01/2015, 12h37

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 :

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
 
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);
}

**Rémi1995** · 18/01/2015, 16h17

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 ...

**diogene** · 18/01/2015, 16h57

Code :

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

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 :

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

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

**Rémi1995** · 18/01/2015, 17h04

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("$-->"); ?

**Rémi1995** · 19/01/2015, 09h17

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.

**imperio** · 19/01/2015, 12h56

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.

**Rémi1995** · 22/01/2015, 14h59

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 :

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
 
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.

**Rémi1995** · 23/01/2015, 18h27

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

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
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);
        }
    }
}

problème execv NULL

C

Discussions similaires

Partager

Partager