Discussion :

[chrtophe]

Bonjour,

J'utilise le code suivant :

Code :

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

1
2
3
4
 
char *path=getcwd(NULL,0);
strcat(path,":");
strcat(path,getenv("PATH"));

Ce code me permet tout simplement d'ajouter le dossier en cours au début de la variable d'environnement path. Cela fonctionne très bien.

Cependant la doc de strcat précise que la chaine de destination doit être suffisamment grande pour contenir l'ajout. Dans mon cas, ma chaine a été allouée par getcwd. Je présume que mon code fonctionne car l'allocation dépasse les besoins. Mais si je ne gère pas, je dois m'attendre à un beau bug.

Je pensais donc avant l'utilisation de strcat utiliser realloc en donnant la taille necessaire. (strlen sur path + strlen de getenv("PATH"))

Merci de vos retours.

[jlliagre]

Voilà comment je ferais en me cantonnant à du code portable:

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
#include <unistd.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
...
 
char *path;
int path_max=pathconf(".", _PC_PATH_MAX);
int size=((path_max==-1)?1024:path_max);
path=malloc(size+1+strlen(getenv("PATH")));
getcwd(path,size);
strcat(path,":");
strcat(path,getenv("PATH"));
printf("%s\n",path);

Un problème est que la longueur du chemin vers le répertoire courant peut être plus grande que _PC_PATH_MAX ...

Voilà une version qui s'adapte à la longueur du nom du répertoire courant:

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
#include <unistd.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <errno.h>
 
...
 
  char *path;
  int size=1024;
  path=malloc(size+1+strlen(getenv("PATH")));
  while(getcwd(path,size)==NULL) {
    if(errno==ERANGE)
    {
      size*=2;
      free(path);
      path=malloc(size+1+strlen(getenv("PATH")));
    }
    else
    {
      exit(1);
    }
  }
  strcat(path,":");
  strcat(path,getenv("PATH"));

Si tu es sûr que ton code s'exécutera avec une libc contenant un getcwd qui supporte un pointeur et une longueur nuls, tu peux faire plus simple:

Code :

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

1
2
3
4
char *path=getcwd(NULL,0);
path=realloc(path,strlen(path)+2+strlen(getenv("PATH")));
strcat(path,":");
strcat(path,getenv("PATH"));

[chrtophe]

Merci.