Discussion :

[therealmancool]

Bonjour tout le monde!

J'essaie de récupérer plusieurs valeurs dans la base de registre avec la fonction RegQueryValueEx.

Voici la fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
BOOL RegValue(HKEY hKeyRoot, LPCTSTR lpSubKey,DWORD Size)
{
	LONG Error;
	Error=RegQueryValueEx(hKeyRoot,lpSubKey,0,0,(LPBYTE)gPath,&Size);
 	if(Error==ERROR_SUCCESS)
	{
  		return(TRUE);
	}
	else
	{
		return(FALSE);
	}
}

Je l'appelle ainsi:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
LONG Error;
char gPath[512];
 
Error = RegOpenKeyEx (HKEY_LOCAL_MACHINE, TEXT("SYSTEM\\CurrentControlSet\\Services\\Disk\\Enum"), 0, KEY_QUERY_VALUE, &hKeyDisks);
   if (Error == ERROR_SUCCESS)
{
...   
   sprintf_s (gDiskName, 32, "%u", DiskNumber);
   Size = sizeof (gPath);
   Error=RegValue(hKeyDisks,gDiskName,Size);
...
}

avec DiskNumber qui est un entier qui va de 0 à n.

Le problème est que la valeur retournée par RegQueryValueEx(hKeyRoot,lpSubKey,0,0,(LPBYTE)gPath,&Size); dans la fonction RegValue est "ERROR_FILE_NOT_FOUND" alors que si je mets TEXT("0") à la place de "lpSubKey", la fonction lit bien la valeur de la clé "0".
Comment faire pour que cet argument soit une variable?

Merci d'avance

[ram-0000]

Tu compiles en quoi Unicode ou MBCS ?

[therealmancool]

En unicode.

[ram-0000]

En unicode.

Ha ? Il n'y a pas de warnings à la compilation ?

Quand je voie cela :

Code :

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

sprintf_s (gDiskName, 32, "%u", DiskNumber);

c'est une chaine de caractère classique ANSI (pas UNICODE je veux dire) que tu prépares et que tu passes à la fonction RegValue().

Comme tu compile en unicode, c'est la fonction unicode de RegQueryValueEx() qui est appelée avec une subkey attendu en unicode (mais en faite construite en ANSI). C'est peut être cela la cause de ton problème.

Quel est le type de gDiskName ?

[therealmancool]

Quel est le type de gDiskName ?

C'est un LPCTSTR en variable global

[ram-0000]

Je pense sincèrement que tu as un gros problème de mélange ANSI/UNICODE
sprintf_s, c'est une fonction ANSI
char gPath[512]; c'est un buffer ANSI

En ANSI, les caractères font un octet, en unicode, les caractères font 2 octets. Donc une chaine de caractères ANSI n'est pas une chaine de caractères UNICODE

De plus, si gDiskName est un LPTCSTR, c'est un pointeur. Y a t'il une allocation mémoire qui est faite pour allouer un bloc de mémoire.

Si tu veux plus d'aide, donne plus de code.

[therealmancool]

En fait c'est un code que j'ai repris et que j'ai modifié.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
HKEY hKeyDisks;
LONG Error;
DWORD Size;
 
Error = RegOpenKeyEx (HKEY_LOCAL_MACHINE, TEXT("SYSTEM\\CurrentControlSet\\Services\\Disk\\Enum"), 0, KEY_QUERY_VALUE, &hKeyDisks);
if (Error == ERROR_SUCCESS)
{
      sprintf_s (gDiskName, 32, "%u", DiskNumber);
      Size = sizeof (gPath);
      Error = RegQueryValueEx ( hKeyDisks,gDiskName,0,0,(LPBYTE)gPath,&Size);
}
...

avec en variables globales:

Code :

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

1
2
3
 
char gPath[512];
char gDiskName[32];

Et à vrai dire ANSI/Unicode ça me dit rien.
Dans les options de compilation y'a marqué unicode.

Je sais que mon problème vient de la variable "gDiskName" mais je ne vois pas comment le résoudre.

[ram-0000]

A priori,

remplacer

Code :

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

1
2
char gPath[512];
char gDiskName[32];

par

Code :

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

1
2
TCHAR gPath[512];
TCHAR gDiskName[32];

Code :

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

sprintf_s (gDiskName, 32, "%u", DiskNumber);

par

Code :

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

_stprintf_s (gDiskName, sizeof(gDiskName) / sizeof(gDiskName[0]), TEXT("%u"), DiskNumber);

et

Code :

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

Size = sizeof (gPath);

par

Code :

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

Size = sizeof (gPath) / sizeof (gPath[0]);

J'espère ne rien avoir oublié. Ces quelques modif on pour but de rendre ton code insensible au type de compilation ANSI ou UNICODE.

Je pense qu'il faudrait que tu trouves un article (j'en ai pas en tête) pour bien comprendre les subtilités et différences car sinon, tu vas reproduire les même erreurs.

[therealmancool]

J'ai fait les modifs que tu m'as indiquées mais ça me marque une erreur à la compilation:

unresolved external symbol __stprintf_s referenced in function

[Médinoc]

et

Code :

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

Size = sizeof (gPath);

par

Code :

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

Size = sizeof (gPath) / sizeof (gPath[0]);

J'ai un doute sur celui-ci.
Si j'ai bien compris la doc, RegQueryValueEx() travaille toujours sur une taille en bytes, pas en TCHARs...

@therealmancool: Es-tu sûr que tu n'as pas encore loupé un warning? Du genre un warning qui te dirait que la fonction _stprintf_s n'est pas déclarée ?
Tu dois inclure <tchar.h>, qui définira _stprintf_s en tant que macro sur indiquant la fonction _swprintf_s().

[ram-0000]

J'ai un doute sur celui-ci.

et

Code :

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

Size = sizeof (gPath);

par

Code :

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

Size = sizeof (gPath) / sizeof (gPath[0]);

Si j'ai bien compris la doc, RegQueryValueEx() travaille toujours sur une taille en bytes, pas en TCHARs...

Tu as tout à fait raison, j'avais pas vu le détail qui tue ... Il faut laisser

Code :

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

Size = sizeof (gPath);