Discussion :

[fregolo52]

Bonjour,

Je galère depuis qq heures pour créer une fenêtre (invisible) dans une DLL.
Mon but est d'avoir un hwnd pour pouvoir utiliser SendMessage pour communiquer avec la DLL.

Pour simplifier, j'ai fait un truc qui ressemble à ca :

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
BOOL WINAPI DllMain(HANDLE hModule, DWORD ul_reason_for_call, LPVOID lpReserved)
{
  switch(ul_reason_for_call)
  {
    case DLL_PROCESS_ATTACH:
      WNDCLASS    wc;
      memset(&wc, 0, sizeof(wc));

      wc.lpfnWndProc   = (WNDPROC)WizardHookWindowDialogProc;
      wc.hInstance     = hModule;
      wc.lpszClassName = L"MaFenetreInvisible";

      ::RegisterClass(&wc);
      HWND dialogWindow = ::CreateWindow(wc.lpszClassName, L"MaFenetreInvisible", WS_ICONIC,
                          0, 0, 0, 0,
                          NULL, NULL, hModule, NULL); 
  }
}

J'ai tenté plusieurs styles, mais en fonction ce ceux-ci, j'ai de beaux crashs. Dans tous les cas j'ai dialogWindow à NULL.
Dans certains cas, j'ai les erreurs ERROR_RESOURCE_TYPE_NOT_FOUND ou ERROR_RESOURCE_NAME_NOT_FOUND ou encore ERROR_SUCCESS (étrange vu que dialogWindow est NULL).

Si quelqu'un pouvait m'aiguiller !!!

Merci

[Médinoc]

Bonjour,
Tu n'es pas censé appeler des fonctions de user32.dll dans DllMain(). Ni d'aucune autre DLL, d'ailleurs, à part Kernel32.

De plus, ce case en WNDPROC est absolument hideux. Si ça ne compile pas sans ce cast, c'est que ta fenêtre n'est pas créée correctement.

PS: Vu l'usage que tu prévois de ta fenêtre, peut-être une "message-only window" suffirait: ajoute HWND_MESSAGE dans le paramètre hwndParent...

[fregolo52]

Ok, j'ai un peu trop simplifié mon exemple.
En fait dans DllMain, je crée un thread et tout le code que tu vois est dans ce thread (avec la boucle de messages).

Ok, je vire le cast.
J'ai fait tellement d'essais qu'à force je fais sûrement n'importe quoi !

Je vais encore investiguer pour savoir pourquoi j'ai un HWND = null et GetLastError() = 0.

[Médinoc]

Peux-tu poster une version moins simplifiée qui reproduit le problème?
Par exemple, que fais-tu dans ton WM_NCCREATE ou WM_CREATE?

[fregolo52]

Bah, en fait je fais que dalle, je poste mon code, avec plein de code commentés liés à mes essais (et j'en ai supprimés)

C'est du code incomplet pour le moment, vu que je tatonne, c'est peut-être ça le souci.

Dans DllMain

Code :

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

1
2
3
4
5
...	case DLL_PROCESS_ATTACH:
		WinLib_Util_ChangeWindowMessageFilter(getHighlightMessage(), MSGFLT_ADD);
		g_WizardHookWindow = new WizardHookWindow(hModule);
_beginthread( LocalWindowThread, 0, (void *) g_WizardHookWindow);
...

Le Thread :

Code :

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

1
2
3
4
5
6
7
8
9
10
void LocalWindowThread (void *pThis)
{
	MSG msg;
	WizardHookWindow *wizardWindow = (WizardHookWindow *)pThis;
	wizardWindow->initWindow();

	// boucle d'évènements
	while( GetMessage(&msg,0,0,0) ) {// reçoit les messages de Windows
	}
}

La classe : attention code pourri !!!

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
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
66
67
68
69
70
71
BOOL WizardHookWindow::RegisterWindow() {
	WNDCLASS wndClass;

    wndClass.style = 0;
	wndClass.lpfnWndProc = WizardHookWindow::WizardHookWindowDialogProc;
    wndClass.cbClsExtra = 0; 
    wndClass.cbWndExtra = 0;
    wndClass.hInstance = GetModuleHandle(NULL);
    wndClass.hIcon = NULL;
    wndClass.hCursor = NULL; 
    wndClass.hbrBackground = NULL;
    wndClass.lpszMenuName = NULL; 
    wndClass.lpszClassName = L"WizardHookWindow";	

	m_comClassAtom = RegisterClass(&wndClass);
	return TRUE;
}

BOOL
WizardHookWindow::initWindow() {

	if (!RegisterWindow())
		return FALSE;
	dialogWindow = NULL;

	dialogWindow = CreateWindow(L"WizardHookWindow", 
		L"WizardHookWindow", 
		WS_OVERLAPPED, 
		CW_USEDEFAULT, 
		CW_USEDEFAULT, 
		CW_USEDEFAULT, 
		CW_USEDEFAULT, 
		HWND_MESSAGE, 
		NULL, 
		GetModuleHandle(NULL), 
		NULL); 
 
/*    WNDCLASS    wc;

    memset(&wc, 0, sizeof(wc));

    wc.lpfnWndProc   = WizardHookWindowDialogProc;
    wc.hInstance     = windowsInstance;
    wc.lpszClassName = L"WizardHookWindow";

    ::RegisterClass(&wc);
    dialogWindow = ::CreateWindow(wc.lpszClassName, NULL, 0,
                          CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT,
                          HWND_MESSAGE, NULL, windowsInstance, NULL); 

*/
	std::wostringstream os;
	cStackBuffer<wchar_t,64> valBuf;

   /* HWND theDialogWindow = CreateDialog(windowsInstance,
                                   L"WizardHookWindow", NULL,
                                   (DLGPROC) WizardHookWindowDialogProc);*/
    // If window could not be created, return "failure".
	if (!dialogWindow) {
		_itow(GetLastError(), valBuf, 10);
		os << L"WizardHookWindow : create dialog failed " << valBuf.c_str() << std::endl;
		::OutputDebugString(os.str().c_str());
        return FALSE;
	}

	_itow((int)dialogWindow, valBuf, 10);
	os << L"WizardHookWindow : window  created " << valBuf.c_str() << std::endl;
	::OutputDebugString(os.str().c_str());

    return TRUE;      
}

et la callback

Code :

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

1
2
3
4
5
6
7
8
9
10
11
LRESULT CALLBACK WizardHookWindow::WizardHookWindowDialogProc (HWND hDlg, UINT message, UINT wParam, LONG lParam) {

    if (message == getHighlightMessage()) {
		            POINT point;
            POINTSTOPOINT(point, lParam);
            processMessage(hDlg, wParam, point);

    }
    return FALSE;   
}

[Médinoc]

Je vois deux problèmes:

1. GetModuleHandle(NULL) retourne systématiquement le handle de l'EXE: Le seul moyen "propre" pour une DLL de connaître son handle, c'est DllMain(). Généralement, je profite du DllMain() pour mémoriser ce handle dans une variable globale à la DLL, et ne plus y toucher après.
2. Ta fonction ressemble à une DLGPROC détournée de son usage. Une WNDPROC doit appeler DefWindowProc() pour son traitement par défaut, pas retourner FALSE.

[fregolo52]

1. En effet je me doutais qu'il y avait un truc dans le genre.

2. En effet, je suis parti d'un code "Dialog".

Après modif, ça marche.

Merci bcp pour ton aide très précieuse.

PS : A trop travailler avec les MFC, on ne sait plus ce que fait win32