explication sur la fonction getenv

Bonjour,

j'ai lu le man mais je n'ai pas trouve ma reponse
est ce que la fonction getenv est limite a un seul appel ou est ce qu'on peut l'appeller plusieurs fois dans un programme

merci :)

Bien sûr que tu peux l'appeler plusieurs fois, je ne vois pas ce qui te fait douter.

Tu peux bien sur l'utiliser autant de fois que tu veux.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

char var[100], * p; p = getenv("myvar"); if (strlen(p) < (size_t)sizeof(var)) { strcpy(var, p); ... }   ...   p = getenv(...);   ...

---

Citation:

---

Envoyé par m0ul3sh0t

Bonjour,

j'ai lu le man mais je n'ai pas trouve ma reponse
est ce que la fonction getenv est limite a un seul appel ou est ce qu'on peut l'appeller plusieurs fois dans un programme

merci :)

---

En fait, le gros problème de getenv() est qu'elle travaille avec une zone statique. En effet, elle
1) commence par remplir une zone statique avec le résultat attendu
2) te renvoie le pointeur de cette zone
Mais la zone est unique. Donc pour deux appels successifs le second écrase le premier.

Exemple

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

int main() { char *home; char *path;   home=getenv("HOME"); path=getenv("PATH");   printf("home=[%s]\n", home); printf("path=[%s]\n", path);   return 0; }

---

Tu t'attends à avoir deux valeurs différentes pour le HOME et le PATH et tu as deux fois le PATH parce que tu affiches à chaque fois la même zone qui a été écrasée et réécrasée lors des deux getenv().

Donc tu peux parfaitement appeler plusieurs fois getenv() mais faut penser, lors de chaque appel, à recopier la zone statique de la librairie dans une zone de ton propre programme.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

int main() { char home[1024]; char path[1024];   strcpy(home, getenv("HOME")); strcpy(path, getenv("PATH"));   printf("home=[%s]\n", home); printf("path=[%s]\n", path);   return 0; }

---

Là ça marche parfaitement (sous réserve qu'aucune chaîne ne dépassera 1024 car.)

Ensuite tu entres dans le problème du multithread où on se demande ce qu'il peut se passer si deux thread appellent chacun getenv() en même temps mais c'est un autre problème plus complexe.

en fait je stockais ce que me renvoie getenv dans un char * que je reutilisais par la suite.
pour le premier appel a gentenv("PATH") j'obtenais tout le path alors que pour le deuxieme appel je n'avais qu'une petite partie, c'est pour ca que je doutais du fait qu'on puisse l'appeller plusieurs fois

Citation:

---

Envoyé par Melem

Code:

---

1 2 3 4 5 6

char var[100], * p; p = getenv("myvar"); if (strlen(p) < (size_t)sizeof(var)) {   ...

---

---

Attention, à ma connaissance, getenv() peut retourner NULL, notamment si la variable n'existe pas :

Code:

---

1 2 3 4 5 6

{ char *p = getenv("myvar"); if (p != NULL) { if (strlen(p) < (size_t)sizeof(var))

---

Citation:

---

Envoyé par Franck.H

Bien sûr que tu peux l'appeler plusieurs fois, je ne vois pas ce qui te fait douter.

---

La variable statique, probablement... Si le pointeur n'est pas NULL, il faut effectivement faire rapidement une copie. strdup() (POSIX.1) peut aider.

Preuve est faite que les statiques internes, c'est le diable !

Citation:

---

Envoyé par m0ul3sh0t

en fait je stockais ce que me renvoie getenv dans un char * que je reutilisais par la suite.
pour le premier appel a gentenv("PATH") j'obtenais tout le path alors que pour le deuxieme appel je n'avais qu'une petite partie, c'est pour ca que je doutais du fait qu'on puisse l'appeller plusieurs fois

---

Tu faisais exactement comme dans mon premier exemple...

Citation:

---

Envoyé par Sve@r

Tu faisais exactement comme dans mon premier exemple...

---

dans ton exemple tu rappelles getenv directement apres
moi j'appelle getenv, jutilise ce que je recupere ET je le rappelle apres pour pouvoir reutiliser ce que je recupere :)

Citation:

---

Code:

---

1 2 3 4 5 6

char *home; char *path;   home=getenv("HOME"); path=getenv("PATH");

---

---

La fonction getenv te renvoi un pointeur vers l'emplacement mémoire auquel se trouve la valeur que tu souhaite lire. Ainsi tu stocke l'adresse de l'emplacement mémoire, pas le contenu qui se trouve physiquement à cette endroit. Tu n'es pas "maître" du contenu de cette variable. Si tu veux récupèrer la valeur et ne plus rien risquer, il te faut la coper vers un emplacement mémoire alloué par ton propre programme, donc plutôt qu'un pointeur comme le disait Svear.

Greg

au final jai opte pour un strdup de ce que me renvoie getenv
je ne voulais pas duppliquer les donnees parce que jen avais besoin pour une fonction mais apres refaire appel a getenv pour pouvoir avoir un programme qui recupere la valeur de mon PATH meme si qqch le fait changer :)
tant pis pour le strdup ^^

Citation:

---

Envoyé par m0ul3sh0t

au final jai opte pour un strdup de ce que me renvoie getenv

---

N'oublie pas que strdup() utilise malloc() et qu'il faut donc libérer le bloc avec free() quand tu n'en as plus besoin.

Citation:

---

Envoyé par m0ul3sh0t

dans ton exemple tu rappelles getenv directement apres
moi j'appelle getenv, jutilise ce que je recupere ET je le rappelle apres pour pouvoir reutiliser ce que je recupere :)

---

Ouaip. Donc si effectivement tu n'as plus besoin du résultat du premier appel lors du second appel, tu peux te passer de sa sauvegarde (je parle de la sauvegarde du premier résultat).
Si au contraire tu en as encore besoin ben faut le conserver par un moyen ou un autre (comme en effet strdup())

Citation:

---

Envoyé par Sve@r

Ouaip. Donc si effectivement tu n'as plus besoin du résultat du premier appel lors du second appel, tu peux te passer de sa sauvegarde (je parle de la sauvegarde du premier résultat).
Si au contraire tu en as encore besoin ben faut le conserver par un moyen ou un autre (comme en effet strdup())

---

bin je n'ai pas besoin de la reutiliser tel quelle etait au debut, jai besoin de re recuperer la valeur du getenv mais je ne recupere qu'une partie de la valeur