Discussion :

[mickael777]

Bonjour à tous ,

Je galère pas mal avec le chargement d'une DLL qui doit être recherchée en fonction de la version de Windows (de NT à XP) dans le dossier "system32". J'utilise pour cela getenv("windir").
Cela marche très bien sur mon PC équipé XP, mais ne fonctionne pas sur d'autre PC équipé d'XP ou 2000.
La chaine de caractère renvoyé par getenv semble être correct puisque je la récupère également dans une clef de registre (différente en fonction de l'OS).
Si je rentre en dur le chemin dansLoadLibrary en fonction XP ou 2000 pas de pb.

Voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
char resultRegister	= 0;
	char* windowsDirectory = (char*)malloc(sizeof(char)*SPACE); 
	windowsDirectory = getenv("windir");
	resultRegister = CreatRegister(windowsDirectory);
strcat(windowsDirectory,PATH);
	if ((hook=LoadLibrary(windowsDirectory)))			
        {...
        }     
	else
	{	
		MessageBox(NULL,"!La DLL n'est pas chargée!","Message d'information",MB_ICONERROR);
         }

Si quelqu'un peut m'aider. Merci

[diogene]

Code :

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

1
2
3
4
	char* windowsDirectory = (char*)malloc(sizeof(char)*SPACE); 
	windowsDirectory = getenv("windir");
	resultRegister = CreatRegister(windowsDirectory);
strcat(windowsDirectory,PATH);

- L'allocation par malloc est inutile.
- Le strcat est illégal, car on n'a pas le droit de modifier la chaîne dont l'adresse est renvoyée par getenv.

[Vincent Rogier]

tu devrais plutot utiliser la variable SystemRoot et tenir compte des remarques de diogene

[Médinoc]

Voire mieux que les variables d'environnement, directement utiliser la fonction GetSystemDirectory(), 100% fiable pour avoir le bon répertoire...

Sachant que si la DLL est déjà dans system32, tu n'as pas besoin de tout ce fratras : Dynamic-Link Library Search Order

[mickael777]

Un grand merci pour vos réponses .

Mon problème est maintenant résolu.
Voici le code utilisé si sa interesse quelqu'un :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
	char resultRegister	= 0;
	char* windowsDirectory = (char*)malloc(sizeof(char)*SPACE); 
		
	GetSystemDirectory(windowsDirectory,SPACE);	
	resultRegister = CreatRegister(windowsDirectory);
	strcat(windowsDirectory, NAME);			
	if ((hook=LoadLibrary(windowsDirectory)))			
	{
        }
	else
	{	
	MessageBox(NULL,"!La DLL n'est pas chargée!","Message d'information",MB_ICONERROR);
         }

J'attend vos remarques, si quelqu'un souhaite en mettre pour le code utilisé.
Je placerai le tag résolu en soirée.

[Médinoc]

Ben, une remarque pour commencer:
On ne caste pas le retour de malloc() en C.

Et aussi, ton code ne compilera pas en mode Unicode.

[mickael777]

Ben, une remarque pour commencer:
On ne caste pas le retour de malloc() en C.

Merci pour t'as réponse.

Une remarque cependant, d'après la FAQ C du site http://c.developpez.com/faq/?page=po...POINTEURS_cast
la cast n'engendre pas de différence. Est-ce vrai ?

[Médinoc]

Aucune différence tant que tu n'oublies pas d'inclure <stdlib.h>.
Si tu oublies de l'inclure, le cast peut masquer cet oubli, menant à des problèmes pouvant être spectaculaires selon la plate-forme.