Discussion :

[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

[Franck.H]

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

[Emmanuel Delahaye]

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 !

[Melem]

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

Code :

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

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

[Emmanuel Delahaye]

Code :

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

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 :

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

1
2
3
4
5
6
 
{
   char *p = getenv("myvar");
   if (p != NULL)
   {
      if (strlen(p) < (size_t)sizeof(var))

[Sve@r]

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 :

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

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 :

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

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.

[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

[Sve@r]

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

[m0ul3sh0t]

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

[greg13]

Code :

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

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

[Sve@r]

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())