Discussion :

[FenX.]

Bonjour à tous

J'ai un souci avec le code suivant :

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
 
std::list<const char *> MyClass::getEnvParametersMatching(const char *keyRegex) 
{
  std::list<const char *> *listRet = new std::list<const char *>();
 
  static regex_t patternBuf;
 
  extern char **environ;
  char strEnviron[256];
  char* pchVarName = NULL;
 
  if ( regcomp( &patternBuf, keyRegex, REG_EXTENDED ) == 0 )
  {
    // Parcours de l'environnement
    while ( *environ != NULL )
    {
    	// Extrait le nom de la variable
    	strcpy( strEnviron, *environ );
    	pchVarName = strtok( strEnviron, "=");
 
	// Si la clé correspond
    	if (regexec( &patternBuf, pchVarName, 0, 0, 0 ) == 0)
    	{	
    	    // On l'ajoute à la list
    	    listRet->push_back( *environ ); // SEG FAULT ???
    	}
 
    	// Variable d'environnement suivante
    	environ++;
    }
  }
 
  return listRet;
}

J'essaie d'insérer *environ ( qui est une chaîne de caractères, n'est-ce pas ? ) dans ma liste, mais j'ai systématiquement une segmentation fault à ce niveau...
S'il est effectivement impossible d'ajouter une chaine "VAR=VAL" dans ma list, est-ce que vous auriez une idée pour ajouter simplement le nom de la variable ( VAR ) dans la list ?

Auriez-vous une idée d'où se trouve le problème ?

Par ailleurs, on m'a dit sur le forum C qu'il était interdit de faire environ++. Alors comment parcourir les variables d'environnement autrement ?

Merci d'avance !

[nikko34]

quitte à utiliser une std::list, pourquoi ne pas utiliser aussi des std::string?

[FenX.]

Pas le choix, imposé par l'historique de l'application

[Laurent Gomila]

Auriez-vous une idée d'où se trouve le problème ?

Une petite, oui :

Code :

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

1
2
std::list<const char *> *
char **

On ne devrait jamais voir ces horreurs dans un code C++. Enfin, si c'est imposé...

Plus précisément, je ne vois pas où tu alloues la mémoire pour environ et *environ. Mais comme c'est une variable globale (beurk... et puis la manière dont elle est déclarée est un peu bourrine), peut-être que tout ceci est fait par une autre fonction.

A part ça, il y a aussi le débogueur qui est là pour t'aider.

[yan]

Ton segfault viens de la

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
  char strEnviron[256];
  char* pchVarName = NULL;

  if ( regcomp( &patternBuf, keyRegex, REG_EXTENDED ) == 0 )
  {
    // Parcours de l'environnement
    while ( *environ != NULL )
    {
    	// Extrait le nom de la variable
    	strcpy( strEnviron, *environ );
    	pchVarName = strtok( strEnviron, "=");

strtok va modifier strEnviron en memoire.
Donc au deuxième passage strcpy( strEnviron, *environ ); va faire n'importequoi

[FenX.]

Effectivement, j'avais pas pensé à ça

Merci beaucoup !

Pour info, voici le parcours des variables d'environnement correct :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  extern char **environ;
  char** ptrEnviron;
 
  ptrEnviron = environ;
 
  // Parcours de l'environnement
  while ( *ptrEnviron != NULL )
  {
  	// blabla
 
  	// Variable d'environnement suivante
  	ptrEnviron++;
  }

[yan]

Ton segfault viens de la

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
  char strEnviron[256];
  char* pchVarName = NULL;

  if ( regcomp( &patternBuf, keyRegex, REG_EXTENDED ) == 0 )
  {
    // Parcours de l'environnement
    while ( *environ != NULL )
    {
    	// Extrait le nom de la variable
    	strcpy( strEnviron, *environ );
    	pchVarName = strtok( strEnviron, "=");

strtok va modifier strEnviron en memoire.
Donc au deuxième passage strcpy( strEnviron, *environ ); va faire n'importequoi

Oups j'ai dit n'importe quoi
- strtok va modifier strEnviron en memoire. ça c'est vrai

- Donc au deuxième passage strcpy( strEnviron, *environ ); va faire n'importequoi; ça c'est faux

Ton segfault viens peut être que char strEnviron[256] est trop petit.
essaie char strEnviron[1024]. Tout façon, ton erreur mémoire doit être avant le push_back, ce qui fait planter le celui ci. ATTENTION, Ce comportement peut être aléatoire. ça peut marcher pendant un temps et du jour au lendemain re planter.