Lecture de la base de registre

Version imprimable

Bonjour,

J'ai besoin dans un programme d'accéder à la base de registre, pour trouver certaines informations sur les clé USB connectées : Vid/Pid, et numéro de série.

J'ai pu voir que ces infos étaient disponibles sur des clé 0, 1, 2, ..... à l'endroit suivant :

Citation:

SYSTEM/CurrentControlSet/Service/USBSTOR/Enum

J'ai écrit un petit programme pour récupérer la valeur de cette clé, mais comme je débute, j'ai une erreur et je ne comprend pas d'où elle vient.
J'utilise le code suivant :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 char* LectureRegistre() { HKEY hk; char *test; LPBYTE valeur=NULL; LPDWORD sizeValeur=NULL; LPCWSTR cle = _T("SYSTEM\\CurrentControlSet\\Service\\USBSTOR\\Enum"); if (RegOpenKeyEx(HKEY_LOCAL_MACHINE, cle, 0, KEY_READ, &hk)) { if(RegQueryValueEx(hk, _T("0"), 0, 0, valeur, sizeValeur)) { sizeValeur=(LPDWORD)sizeof(valeur); if(RegQueryValueEx(hk, _T("0"), 0, 0, valeur, sizeValeur)) { test=(char*)*valeur; } } } RegCloseKey(hk); return test; }
L'erreur se produit lors de l'exécution, au niveau de la ligne test=(char*)*valeur;

Citation:

Exception non gérée à 0x0041392b dans CryptageCle.exe : 0xC0000005: Violation d'accès lors de la lecture de l'emplacement 0x00000000.

Il semblerai donc que le pointeur valeur ne reçoivent pas l'adresse de cette clé.

Merci d'avance pour votre aide

En fait, j'avais oublié un s à Services dans le chemin de la clé. Je l'ai vu en rajoutant des ERROR_SUCCESS à mes if (qui était apparemment validés à chaque fois).

Par contre, le pointeur valeur reste toujours à l'adresse 0x00000000. Je n'arrive pas à comprendre pourquoi.

Je laisse le nouveau code, si besoin :
Code:

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 35 36 37 38 char* LectureRegistre() { HKEY hk; BYTE test2; char *data = "Erreur"; LPBYTE valeur=NULL; LPDWORD sizeValeur=NULL; LPCWSTR cle = _T("SYSTEM\\CurrentControlSet\\Services\\USBSTOR\\Enum"); if (RegOpenKeyEx(HKEY_LOCAL_MACHINE, cle, 0, KEY_READ, &hk) == ERROR_SUCCESS) { if(RegQueryValueEx(hk, _T("0"), 0, 0, valeur, sizeValeur) == ERROR_SUCCESS) { sizeValeur=(LPDWORD)sizeof(valeur); if(RegQueryValueEx(hk, _T("0"), 0, 0, valeur, sizeValeur) == ERROR_SUCCESS) { data=(char*)*valeur; } else { MessageBox(0,_T("Erreur de lecture de la clé registre contenant les infos sur la clé USB"),NULL,0); } } else { MessageBox(0,_T("Erreur de lecture de la clé registre contenant les infos sur la clé USB (2)"),NULL,0); } } else { MessageBox(0,_T("Erreur d'ouverture de la base de registre pour lire les infos sur la clé"),NULL,0); } RegCloseKey(hk); return data; }
Le débogage me dit :

Citation:

+ &hk 0x0012f318 HKEY__ * *
+ hk 0x00000ed0 {unused=??? } HKEY__ *
+ valeur 0x00000000 <Ptr> incorrect unsigned char *

Sinon, j'ai oublié de préciser, je sais pas si c'est important : je suis sous windows et j'utilise visual c++ 2005

10/04/2008, 21h53
ram-0000
Allez, je me lance dans la réponse

L'appel à RegOpenKeyEx() me semble correct par contre, ca coince du côté de RegQueryValueEx():?

Ton code:
Code:

1 2 3 4 LPBYTE valeur=NULL; LPDWORD sizeValeur=NULL; if(RegQueryValueEx(hk, _T("0"), 0, 0, valeur, sizeValeur) == ERROR_SUCCESS)
Mon code:
Code:

1 2 3 4 5 BYTE valeur[256]; DWORD sizeValeur=sizeof(valeur); DWORD Type; if(RegQueryValueEx(hk, _T("0"), 0, &Type, valeur, &sizeValeur) == ERROR_SUCCESS)
De plus, je ne sais pas ton mode de compilation Unicode ou Mbcs, c'est très important.

le code "LCWSTR cle = _T("SYSTEM..." me fait croire que c'est du unicode et donc si c'est cela, la string retournée par l'appel RegQueryValueEx() sera aussi une chaine unicode. Tu ne peux pas et ne doit pas la retourner en tant que char * (code de retour de ta fonction)

Il va falloir passer par un appel à WideCharToMultiByte().

D'ailleur, à ce sujet, si tu es en Unicode, que fait une fonction retournant un char * ?

Le pointeur valeur, tu dois l'allouer toi-même.
Ce code corrigé devrait retourner NULL en cas d'échec, ou un pointeur générique vers les données de la clé en cas de succès.
Ce pointeur devra être libéré en appelant la fonction FreeRegistre() quand tu n'en auras plus besoin.
Note: Si la valeur est de type REG_SZ, les données seront un tableau de TCHAR.

Code:

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
35
36
37
38
39
40
41
42
43
44
45
void* AllocRegistre(size_t taille)
{
	return new char[taille];
}
 
void FreeRegistre(void*pv)
{
	char* p = static_cast< char* >(pv);
	delete[] p;
}
 
void* LectureRegistre()
{
	HKEY hk = NULL;
	LPBYTE pValeur=NULL;
	DWORD sizeValeur=0; /* Pas un pointeur ici */
	LPCTSTR const cle = _T("SYSTEM\\CurrentControlSet\\Services\\USBSTOR\\Enum");
 
	if (RegOpenKeyEx(HKEY_LOCAL_MACHINE, cle, 0, KEY_READ, &hk) == ERROR_SUCCESS)
	{
		if(RegQueryValueEx(hk,  _T("0"), NULL, NULL, NULL, &sizeValeur) == ERROR_SUCCESS)
		{
			pValeur = static_cast< LPBYTE >(AllocRegistre(sizeValeur));
			if(RegQueryValueEx(hk,  _T("0"), NULL, NULL, pValeur, &sizeValeur) == ERROR_SUCCESS)
			{
				// OK!
			}
			else
			{
				MessageBox(NULL, _T("Erreur de lecture de la clé registre contenant les infos sur la clé USB"), NULL, MB_OK|MB_ICONERROR);
				FreeRegistre(pValeur), pValeur=NULL;
			}
		}
		else
		{
			MessageBox(NULL, _T("Erreur de lecture de la clé registre contenant les infos sur la clé USB (2)"), NULL, MB_OK|MB_ICONERROR);
		}
		RegCloseKey(hk);
	}
	else
	{
		MessageBox(NULL, _T("Erreur d'ouverture de la base de registre pour lire les infos sur la clé"), NULL, MB_OK|MB_ICONERROR);
	}
	return pValeur;
}

Au sujet du LPDWORD: http://blogs.msdn.com/oldnewthing/ar...28/508689.aspx

11/04/2008, 12h51
CyberSlan

Merci pour vos réponses.

Je test ça dans l'après midi, et je vous dis ce que ça donne.

Pour le mode de compilation, je sais pas ce que j'utilise. J'ai créé un projet MFC et je n'ai pas touché aux paramètres de compilation. Où est ce que je peux voir ça?

Sinon, cette fonction retourne un char car je veux traiter le résultat, et le mettre dans un fichier texte, mais ce n'est peut être pas la meilleur solution.

Le problème pour "retourner un char", c'est que RegQueryValueEx() retourne des TCHAR, donc il y aura (ou pas, selon le mode de compilation) une conversion à faire.
Vu que le nom de la valeur est hard-codé, je te conseille d'utiliser explicitement RegQueryValueExA(). Et peut-être réjouter du code pour être sûr que la variable est bien de type REG_SZ...

Code:

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
35
36
37
38
39
40
41
42
43
44
45
46
void* AllocRegistre(size_t taille)
{
	return new char[taille];
}
 
void FreeRegistre(void*pv)
{
	char* p = static_cast< char* >(pv);
	delete[] p;
}
 
char* LectureRegistre()
{
	HKEY hk = NULL;
	LPBYTE pValeur=NULL;
	LPCTSTR const cle = _T("SYSTEM\\CurrentControlSet\\Services\\USBSTOR\\Enum");
 
	if (RegOpenKeyEx(HKEY_LOCAL_MACHINE, cle, 0, KEY_READ, &hk) == ERROR_SUCCESS)
	{
		DWORD sizeValeur=0; /* Pas un pointeur ici */
		DWORD type = 0;
		if(RegQueryValueExA(hk,  "0", NULL, &type, NULL, &sizeValeur) == ERROR_SUCCESS && type==REG_SZ)
		{
			pValeur = static_cast< LPBYTE >(AllocRegistre(sizeValeur));
			if(RegQueryValueExA(hk,  "0", NULL, NULL, pValeur, &sizeValeur) == ERROR_SUCCESS)
			{
				// OK!
			}
			else
			{
				MessageBox(NULL, _T("Erreur de lecture de la clé registre contenant les infos sur la clé USB"), NULL, MB_OK|MB_ICONERROR);
				FreeRegistre(pValeur), pValeur=NULL;
			}
		}
		else
		{
			MessageBox(NULL, _T("Erreur de lecture de la clé registre contenant les infos sur la clé USB (2)"), NULL, MB_OK|MB_ICONERROR);
		}
		RegCloseKey(hk);
	}
	else
	{
		MessageBox(NULL, _T("Erreur d'ouverture de la base de registre pour lire les infos sur la clé"), NULL, MB_OK|MB_ICONERROR);
	}
	return pValeur;
}

11/04/2008, 14h13
CyberSlan

Re-bonjour,

J'ai essayé les deux solutions que vous m'avez proposé, mais je n'arrive toujours pas à obtenir la chaîne que je veux. L'erreur d'exécution est corrigée, et j'arrive à obtenir quelque chose dans mon fichier de sortie :

Avec le code de ram_0000, j'obtiens la chaîne suivante :

Citation:

ÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÀõ ÌÌÌÌÌÌÌÌÌÌÌ ÌÌÌÌÌÌÌÌü ÌÌÌÌÌÌÌÌ

Avec celui de Médinoc, j'obtiens la première lettre de la chaîne stockée dans le registre : U. j'ai essayé avec d'autres clés registre, et cela correspond bien à la première lettre de la chaîne de caractères

Je devrai obtenir la chaîne suivante (de type REG_SZ) :

Citation:

USB\Vid_0930&Pid_6533\0DA0C560B370CA66

J'ai bien vu :

Citation:

Note: Si la valeur est de type REG_SZ, les données seront un tableau de TCHAR.

mais lorsque je cast en TCHAR, je n'obtient que la valeur héxadécimal de la lettre que j'avais avant.

J'ai essayé plusieurs codes pour récupérer les lettres suivantes, mais j'ai une erreur à chaque fois... J'avoue que je n'ai pas vraiment compris le fonctionnement du code de Médinoc, ce qui m'empêche de trouver la solution moi même :

-Pour moi, une fonction de la forme void nomFonction() ne renvoie rien.
-pValeur est un pointeur, et devrait donc être une adresse héxadécimale, mais quand je fais valeur=(char*)LectureRegistre(); je devrai avoir une adresse dans valeur.

Est ce que quelqu'un peut m'expliquer ça?

Merci d'avance

edit : Désolé, j'avais pas vu ton dernier message Médinoc, peut être que ça va m'aider. Je m'y met tout de suite
11/04/2008, 14h29
CyberSlan

Ça marche.

Merci beaucoup pour ton aide.

Juste une dernière question : une fois que j'ai appelé la fonction lectureRegistre dans le programme principal, je dois appeler FreeRegistre() pour libérer le pointeur, c'est bien ça?

Merci encore, ça fait un petit moment que j'essayais d'obtenir ces infos sans succès, et grâce à toi, j'ai pu me débloquer.
11/04/2008, 14h46
Médinoc

Citation:

Envoyé par CyberSlan

Juste une dernière question : une fois que j'ai appelé la fonction lectureRegistre dans le programme principal, je dois appeler FreeRegistre() pour libérer le pointeur, c'est bien ça?

Oui, c'est bien ça.

Citation:

Envoyé par CyberSlan

Ça marche.

Merci beaucoup pour ton aide.

<snip>

Merci encore, ça fait un petit moment que j'essayais d'obtenir ces infos sans succès, et grâce à toi, j'ai pu me débloquer.

De rien ;)