Problème : erreur de segmentation.

Version imprimable

Bonjour à tous.
J'utilise une class externe dans mon programme (nommé "UsbHID") qui me permet d'accéder à un périphérique USB.

Voici le header :

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
47
48
49
50
51
52
53
54
#if defined(WIN32)

#include <windows.h>
#include <setupapi.h>
extern "C" {
#include <hidsdi.h>
}

#endif

namespace CyberGarage {

class UsbHID
{
private:

#if defined(WIN32)
	SP_DEVICE_INTERFACE_DETAIL_DATA *mDevIfDetailData;
	HANDLE mDevHandle;
	HIDD_ATTRIBUTES mDevAttr;
	HIDP_CAPS mDevCaps;
	HANDLE  mDevEvent;
	OVERLAPPED mOverLapped;
	unsigned char *mInputBuf;
	unsigned char *mOutputBuf;
#endif

public:

	UsbHID();
	virtual ~UsbHID();

	bool open(int venderID, int productID);
	bool close();

	bool isConnected() 
	{
		return (mDevHandle ? true : false);
	}

	int read();
	int write(unsigned char *data, int dataLen);

    //...Code inutile...

#if defined(WIN32)
	void GetLastError();
#endif
};

}

#endif

J'ai remarqué que la fonction qui me pose problème est "open()" : elle me provoque une erreur de segmentation.
Voici le rapport du debugger

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#0 0000D514	??() (??:??)
#1 00401508	??() (??:??)
#2 0022FED0	??() (??:??)
#3 0022FC1C	??() (??:??)
#4 41416161	??() (??:??)
#5 0022FE6C	??() (??:??)
#6 7C91EE18	strchr() (C:\WINDOWS\system32\ntdll.dll:??)
#7 7C9206F0	ntdll!RtlAppendStringToString() (C:\WINDOWS\system32\ntdll.dll:??)
#8 FFFFFFFF	??() (??:??)
#9 7C9206EB	ntdll!RtlAppendStringToString() (C:\WINDOWS\system32\ntdll.dll:??)
#10 77BFC3C9	msvcrt!free() (C:\WINDOWS\system32\msvcrt.dll:??)
#11 003D0000	??() (??:??)
#12 40000060	??() (??:??)
#13 77BFC3CE	msvcrt!free() (C:\WINDOWS\system32\msvcrt.dll:??)
#14 00000000	??() (??:??)

Pourtant d'après le développeur, ça version est stable.
Donc je ne voie pas pourquoi cette erreur survient ??

Code de la fonction open() qui est plutôt costaud :D

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
bool UsbHID::open(int venderID, int productID)
{
	close();
 
	GUID guid;
	HidD_GetHidGuid(&guid);
 
	HDEVINFO devInfoSet = SetupDiGetClassDevs(&guid, 0, 0, (DIGCF_PRESENT | DIGCF_DEVICEINTERFACE ));
	if (devInfoSet == INVALID_HANDLE_VALUE)
		return false;
 
	SP_DEVICE_INTERFACE_DATA devIf;
	devIf.cbSize = sizeof(SP_DEVICE_INTERFACE_DATA);
	for (int i=0; SetupDiEnumDeviceInterfaces(devInfoSet, NULL, &guid, i, &devIf); i++) {
		DWORD devIfDetailSize = 0;
		SetupDiGetDeviceInterfaceDetail(devInfoSet, &devIf, NULL, 0, &devIfDetailSize, 0);
		if (devIfDetailSize <=0)
			continue;
		SP_DEVICE_INTERFACE_DETAIL_DATA *devIfDetail = (SP_DEVICE_INTERFACE_DETAIL_DATA *)malloc(devIfDetailSize);
		devIfDetail->cbSize = sizeof(SP_INTERFACE_DEVICE_DETAIL_DATA);
		if (SetupDiGetDeviceInterfaceDetail(devInfoSet, &devIf, devIfDetail, devIfDetailSize, &devIfDetailSize, 0) != 0) {
			HANDLE devHandle = CreateFile(devIfDetail->DevicePath, GENERIC_READ|GENERIC_WRITE, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL);
			if(devHandle == INVALID_HANDLE_VALUE) {
				free(devIfDetail);
				continue;
			}
			HIDD_ATTRIBUTES devAttr;
			if (!HidD_GetAttributes(devHandle, &devAttr)) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			PHIDP_PREPARSED_DATA devPreData;
			if (!HidD_GetPreparsedData(devHandle, &devPreData )) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			HIDP_CAPS devCaps;
			if (HidP_GetCaps(devPreData, &devCaps ) != HIDP_STATUS_SUCCESS ) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			CloseHandle(devHandle);
			if (devAttr.VendorID == venderID && devAttr.ProductID == productID) {
				mDevHandle = CreateFile(devIfDetail->DevicePath, GENERIC_READ|GENERIC_WRITE, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, FILE_FLAG_OVERLAPPED, NULL);
				SetupDiDestroyDeviceInfoList(devInfoSet);
				memcpy(&mDevAttr, &devAttr, sizeof(HIDD_ATTRIBUTES));
				memcpy(&mDevCaps, &devCaps, sizeof(HIDP_CAPS));
				mInputBuf = (unsigned char *)malloc(mDevCaps.InputReportByteLength);
				mOutputBuf = (unsigned char *)malloc(mDevCaps.OutputReportByteLength);
				return true;
			}
		}
		free(devIfDetail);
	}
 
	SetupDiDestroyDeviceInfoList(devInfoSet);
 
	return false;
}

01/10/2008, 12h49
coyotte507

Salut,

quel debugger utilises-tu?

Dans tous les cas, si tu désactive les optimisations lors de la compilation, et que tu actives les options de debuggage ( profile code, debugging symbols), tout ça dans les options de compilation, tu auras des informations beaucoup plus précises sur le plantage.
01/10/2008, 14h16
bastnt

Bah j'utilise le tout simple debugger de code::blocks.
01/10/2008, 14h28
deadalnix

Fait nous un vrai rapport de bug : ou se produit l'erreur exactement (ou va pas faire le boulot a ta place) le stack trace avec les noms des fonctions, c'est mieux, avec un -Wall dans les paramètres de compilation, ca donne des infos ?

Bref, aide toi toi même en nous aidant.
01/10/2008, 14h30
bastnt
Je viens d'avancer dans mon erreur : après une série de plusieurs petit tests,
J'ai trouvé la fonction à la base de l'erreur (dans la fonction open() )
Code:

1 2 3 4 5 6 7 8 9 bool UsbHID::open(int venderID, int productID) { close(); GUID guid; HidD_GetHidGuid(&guid); // Code sans problème }
et j'ai trouvé son prototype :
Code:

1 2 3 4 void __stdcall HidD_GetHidGuid ( OUT LPGUID HidGuid );
01/10/2008, 14h36
deadalnix

QUE TU DIS LA COMPILATION AVEC WALL ???

QUE DONNE LE STACK TRACE AVEC LES NOMS A LA PLACE DES ???? (Options de compilation).

D'ou sort cette fonction HidD_GetHidGuid ? Est-elle fiable ?
01/10/2008, 14h42
bastnt

Merci de prendre du temps à mon problème deadalnix.

Alors j'ai bien mis le paramètre -Wall dans "debugger initialization commands", mais ça ne change rien au rapport. Je pense que ça vient du fait que la fonction se trouve dans un .a

Oui elle devrait être fiable, car elle proviens de la DDK (- Windows Driver Development Kit)
01/10/2008, 15h38
loufoque

Sinon tu peux essayer de faire tourner ton programme avec valgrind, ça aide bien.
Mais ça marche que sous linux, donc ça risque d'être difficile à compiler, même en utilisant libwine.
01/10/2008, 16h25
deadalnix

C'est étrange. As-tu la définition de GUID ?

Si ceci n'est pas un classe a toi, peut-etre contient-elle un pointeur qui n'est pas initialisé. Peut être devrais-tu essayer de lancer le constructeur par defaut.
01/10/2008, 18h54
coyotte507

Salut,

comme je te l'ai dit dans mon premier post, enlève les optimisations (décoche -O2), rajoute les options de debug (-g -pg), et décoche la case -s (strip all symbols).

Avec ça tu auras un bon rapport de débug ;)
01/10/2008, 19h17
bastnt

loufoque : désolé mais je n'ai pas linux :/

deadalnix : j'ai pas trouvé la definition de GUID dans les headers, mais j'ai trouvé ça sur msdn : http://msdn.microsoft.com/en-us/library/ms790927.aspx
(et si tu clique sur GUID dans le texte, il te donne ça définition; mais moi, ça ne m'éclaire pas trop)
01/10/2008, 19h23
bastnt

coyote : je ne trouves pas tous les paramètres que tu m'as indiqué.
J'ai juste réussi à ajouter les autres, mais ça ne change rien.

http://www.zimagez.com/miniature/screen277.bmp
01/10/2008, 20h08
coyotte507

Non c'est dans les options de compilations (là où y'a le Wall et tout) pas dans les debugger settings ;)
02/10/2008, 19h35
bastnt

Ça y est, j'ai bien trouvé les options dont tu parlais.
Mais cela n'a rien changer au rapport final :/
02/10/2008, 20h57
coyotte507

Bizarre...

Tu-as bien "rebuild" (ctrl+F9)?

Je ne sais pas si tu sais, mais pour changer les options de compilation, il ne faut pas aller dans settings->compiler & debugger, mais clic croit sur le projet->build options, et là tu coches/décoches les cases citées...

Si au pire t'arrives toujours pas à débugger, tu peux uploader ton projet pour qu'on teste..
03/10/2008, 18h53
bastnt

Cette fois jai bien changé aux bons endroits (-g -pg...), mais je ne vois pas de grands changements :/

J'ai mis ici un zip de mon projet code::blocks.
03/10/2008, 20h34
coyotte507

Salut,

je t'avais dit de décocher les cases d'optimisation (-O2) et strip all symbols (-s)

Je ne peux pas tester, il n'y a pas le fichier Wiimote.h.
04/10/2008, 12h49
bastnt
Je crois que wiimote.h est inutile ici, mais je te l'ai quand même uploadé.

Avec les paramètres changés, j'ai un nouveau log :
Code:

1 2 3 #0 0000E69C ??() (??:??) #1 0040152F CyberGarage::UsbHID::open(this=0x3d25a0, venderID=1406, productID=774) (UsbHID.cpp:52) #2 00401EA3 main(argc=1, argv=0x3d2470) (main.cpp:10)
Et c'est bien ce que je soupçonnais, à la ligne 52 : c'est la fonction HidD_GetHidGuid(&guid); qui provoque l'erreur.
(dans un post précédent j'ai mis des liens la concernant : je ne voie pas de problème particulier oO)

Salut,

j'ai enfin réussi à compiler :)

Le programme me dit "Wiimote introuvable"

Ce qui veut dire que la fonction open a retourné false.

Or:

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
bool UsbHID::open(int venderID, int productID)
{
	close();
 
	GUID guid;
	HidD_GetHidGuid(&guid);
 
	HDEVINFO devInfoSet = SetupDiGetClassDevs(&guid, 0, 0, (DIGCF_PRESENT | DIGCF_DEVICEINTERFACE ));
        //Ici se trouve le premier "return false"
	if (devInfoSet == INVALID_HANDLE_VALUE)
		return false;
 
	SP_DEVICE_INTERFACE_DATA devIf;
	devIf.cbSize = sizeof(SP_DEVICE_INTERFACE_DATA);
	for (int i=0; SetupDiEnumDeviceInterfaces(devInfoSet, NULL, &guid, i, &devIf); i++) {
		DWORD devIfDetailSize = 0;
		SetupDiGetDeviceInterfaceDetail(devInfoSet, &devIf, NULL, 0, &devIfDetailSize, 0);
		if (devIfDetailSize <=0)
			continue;
		SP_DEVICE_INTERFACE_DETAIL_DATA *devIfDetail = (SP_DEVICE_INTERFACE_DETAIL_DATA *)malloc(devIfDetailSize);
		devIfDetail->cbSize = sizeof(SP_INTERFACE_DEVICE_DETAIL_DATA);
		if (SetupDiGetDeviceInterfaceDetail(devInfoSet, &devIf, devIfDetail, devIfDetailSize, &devIfDetailSize, 0) != 0) {
			HANDLE devHandle = CreateFile(devIfDetail->DevicePath, GENERIC_READ|GENERIC_WRITE, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL);
			if(devHandle == INVALID_HANDLE_VALUE) {
				free(devIfDetail);
				continue;
			}
			HIDD_ATTRIBUTES devAttr;
			if (!HidD_GetAttributes(devHandle, &devAttr)) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			PHIDP_PREPARSED_DATA devPreData;
			if (!HidD_GetPreparsedData(devHandle, &devPreData )) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			HIDP_CAPS devCaps;
			if (HidP_GetCaps(devPreData, &devCaps ) != HIDP_STATUS_SUCCESS ) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			CloseHandle(devHandle);
			if (devAttr.VendorID == venderID && devAttr.ProductID == productID) {
				mDevHandle = CreateFile(devIfDetail->DevicePath, GENERIC_READ|GENERIC_WRITE, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, FILE_FLAG_OVERLAPPED, NULL);
				SetupDiDestroyDeviceInfoList(devInfoSet);
				memcpy(&mDevAttr, &devAttr, sizeof(HIDD_ATTRIBUTES));
				memcpy(&mDevCaps, &devCaps, sizeof(HIDP_CAPS));
				mInputBuf = (unsigned char *)malloc(mDevCaps.InputReportByteLength);
				mOutputBuf = (unsigned char *)malloc(mDevCaps.OutputReportByteLength);
				return true;
			}
		}
		free(devIfDetail);
	}
 
	SetupDiDestroyDeviceInfoList(devInfoSet);
 
	return false;
}

Comme tu le vois, si la fonction a retourné false sans encombres pour moi, ça veut dire qu'elle bugge plus tard. (la fonction HidD_GetHidGuid() n'est donc pas en cause ;))

Refais des tests à coups de "fprintf" vers stdlog (écriture immédiate, pas dans le tampon), pour voir où la fonction bugge vraiment.

Genre:

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
bool UsbHID::open(int venderID, int productID)
{
	close();
 
	GUID guid;
        fprintf(stdlog, "Appel à HidD_GetHidGuid\n");
	HidD_GetHidGuid(&guid);
        fprintf(stdlog, "Appel à SetupDiGetClassDevs\n");
	HDEVINFO devInfoSet = SetupDiGetClassDevs(&guid, 0, 0, (DIGCF_PRESENT | DIGCF_DEVICEINTERFACE ));
	if (devInfoSet == INVALID_HANDLE_VALUE)
		return false;
 
	SP_DEVICE_INTERFACE_DATA devIf;
	devIf.cbSize = sizeof(SP_DEVICE_INTERFACE_DATA);
        fprintf(stdlog, "Début de la boucle for\n");
	for (int i=0; SetupDiEnumDeviceInterfaces(devInfoSet, NULL, &guid, i, &devIf); i++) {
                fprintf(stdlog, "Itération\n");
		DWORD devIfDetailSize = 0;
		SetupDiGetDeviceInterfaceDetail(devInfoSet, &devIf, NULL, 0, &devIfDetailSize, 0);
		if (devIfDetailSize <=0)
			continue;
		SP_DEVICE_INTERFACE_DETAIL_DATA *devIfDetail = (SP_DEVICE_INTERFACE_DETAIL_DATA *)malloc(devIfDetailSize);
		devIfDetail->cbSize = sizeof(SP_INTERFACE_DEVICE_DETAIL_DATA);
		if (SetupDiGetDeviceInterfaceDetail(devInfoSet, &devIf, devIfDetail, devIfDetailSize, &devIfDetailSize, 0) != 0) {
			HANDLE devHandle = CreateFile(devIfDetail->DevicePath, GENERIC_READ|GENERIC_WRITE, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL);
			if(devHandle == INVALID_HANDLE_VALUE) {
				free(devIfDetail);
				continue;
			}
			HIDD_ATTRIBUTES devAttr;
			if (!HidD_GetAttributes(devHandle, &devAttr)) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			PHIDP_PREPARSED_DATA devPreData;
			if (!HidD_GetPreparsedData(devHandle, &devPreData )) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			HIDP_CAPS devCaps;
			if (HidP_GetCaps(devPreData, &devCaps ) != HIDP_STATUS_SUCCESS ) {
				free(devIfDetail);
				CloseHandle(devHandle);
				continue;
			}
			CloseHandle(devHandle);
			if (devAttr.VendorID == venderID && devAttr.ProductID == productID) {
				mDevHandle = CreateFile(devIfDetail->DevicePath, GENERIC_READ|GENERIC_WRITE, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, FILE_FLAG_OVERLAPPED, NULL);
				SetupDiDestroyDeviceInfoList(devInfoSet);
				memcpy(&mDevAttr, &devAttr, sizeof(HIDD_ATTRIBUTES));
				memcpy(&mDevCaps, &devCaps, sizeof(HIDP_CAPS));
				mInputBuf = (unsigned char *)malloc(mDevCaps.InputReportByteLength);
				mOutputBuf = (unsigned char *)malloc(mDevCaps.OutputReportByteLength);
				return true;
			}
		}
		free(devIfDetail);
	}
 
	SetupDiDestroyDeviceInfoList(devInfoSet);
 
	return false;
}

Et en continuant à logger presque chaque ligne, tu verras à quelle ligne exactement ça plante.

Il te met wiimote introuvable ?
Moi je n'ai même pas ce "printf", l'erreur de segmentation se fait avant.

Et puis j'ai testé avec des printf a chaque ligne dans :
Code:

1 2 3 4 5 6 7 8 9 10 11 bool UsbHID::open(int venderID, int productID) { close(); printf("creation de lobjet guid\n"); GUID guid; printf("get guid\n"); HidD_GetHidGuid(&guid); printf("creation de lobjet devinfo\n"); HDEVINFO devInfoSet = SetupDiGetClassDevs(&guid, 0, 0, (DIGCF_PRESENT | DIGCF_DEVICEINTERFACE )); if (devInfoSet == INVALID_HANDLE_VALUE) return false;
Et il ne s'affiche que :
Code:

1 2 creation de lobjet guid get guid
Donc l'erreur se trouve dans gethidguid, non ?

04/10/2008, 15h06
coyotte507

Sinon...

Dans les options de link de ton projet, j'ai vu que tu as linké les fichier en rajoutant .lib derrière, ça m'avait provoqué une erreur lors de la compilation...

Dans build-options/linker settings, enlève les "../xxx.lib" (ça fait moche), et dans properties->project build options->linker settings, enlève les ".lib" à la fin des fichiers.

J'ai vu là qu'un mauvais linkage pouvait causer ce genre d'erreurs.

edit: j'ai joint mon projet codeblocks
puis build->rebuild pour prendre en compte les modifications.
05/10/2008, 17h04
bastnt

Merci beaucoup coyotte507, j'ai changé ce que tu as dit : et ça affiche bien "wiimote introuble". Plus qu'à connecté la wiimote et c'est bon :)

(Je comprends pourquoi retirer le .lib evite une erreur de segmentation OO)

Merci beaucoup.
(en ésperant que ça marchera)
05/10/2008, 18h00
bastnt

Ca marche avec la wiimote !!!!!!!!!!!!!!!!!

Merci merci merci merci merci merci beaucoup coyote !! :D
pour le temps que tu as pris à mon problème.
05/10/2008, 19h01
coyotte507

Citation:

Envoyé par bastnt

Merci beaucoup coyotte507, j'ai changé ce que tu as dit : et ça affiche bien "wiimote introuble". Plus qu'à connecté la wiimote et c'est bon :)

(Je comprends pourquoi retirer le .lib evite une erreur de segmentation OO)

Merci beaucoup.
(en ésperant que ça marchera)

Ca fait plaisir d'être remercié comme ça lol

En fait si tu rajoutes .lib (par ex hdi.lib) il va chercher hdi.lib.lib. Si tu utilises la version stable la plus récente de Codeblocks (8.02) il aurait dû t'afficher le message d'erreur. (Ou alors utilises-tu un autre compilateur que minGW?)

Bonne chance pour la wiimote!