Discussion :

[Rémi1995]

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]

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]

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

[diogene]

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]

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]

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]

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]

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.