Discussion :

[rdpdo]

Bonjour,

J'ai un probleme de mémoire avec le code de ma DLL ci-dessous, mais je ne trouve pas où il se situe... En fait c'est une DLL appelée via un javascript depuis un gadget windows. Et lors de l'appel à la DLL le debogger de visual studio me met une erreur de mémoire.

A priori ca vient du sprintf car des que je l'utilise ca me met des erreurs, sinon il n'y en a pas...

Si vous avez une idée

Merci de votre aide.

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
 
STDMETHODIMP CclsDLLMSN::SendMessageMSN(BSTR m_strMessage, SHORT show)
{
		HWND msnui = NULL;
		COPYDATASTRUCT msndata;
 
		// ASCII
		char Title[256];
		memset(Title,'\0',sizeof(Title));
 
		unsigned long length = WideCharToMultiByte (CP_ACP,0, m_strMessage,SysStringLen(m_strMessage), NULL, 0,NULL, NULL);
		char* strRet = new char[length];
		length = WideCharToMultiByte (CP_ACP,0, m_strMessage,SysStringLen(m_strMessage), reinterpret_cast <char *>(strRet), length,NULL, NULL);
		strRet[length] = '\0';
			sprintf(Title,"\\0Music\\0%i\\0{0}\\0%s\\0\\0\\0\\0\0",show,strRet);
 
 
		// UTF16
		WCHAR wstrTitle[256];
		memset(wstrTitle, 0, sizeof(CHAR)*256);
 
		// Conversion ASCII vers UTF16
		int nLen = MultiByteToWideChar(CP_ACP, 0, (LPCSTR)Title, -1, NULL, NULL);
		MultiByteToWideChar(CP_ACP, 0, (LPCSTR)Title, -1, (LPWSTR)wstrTitle, nLen);
		msndata.dwData = 0x547;
		msndata.lpData = (void*)wstrTitle;
		msndata.cbData = (nLen*2)+2;
 
		do
		{
			USES_CONVERSION;
			msnui = ::FindWindowEx(NULL, msnui,A2W("MsnMsgrUIManager"), NULL);
			if (msnui != NULL)
				::SendMessage(msnui, WM_COPYDATA, (WPARAM)NULL, (LPARAM)&msndata);
		}while(msnui !=NULL);
		delete strRet;
		return S_OK;
}

[Médinoc]

1. Pas de "m_" pour une variable qui n'est pas une variable membre.
2. Tu alloues length caractères, mais tu places un caractère nul en length+1ième position...
3. Tu n'as aucun moyen d'être sûr que ton sprintf() ne débordera pas du buffer title. Tu dois l'allouer dynamiquement également.
4. Je ne suis pas sûr que les casts explcites de title et wstrTitle servent à grand-chose...
5. D'une part, ce n'est pas A2W qu'il faut utiliser mais A2T. D'autre part, pourquoi utiliser une conversion run-time plutôt que d'utiliser tout simplement la macro _T() ou TEXT() ?
6. Ta boucle do-while() ne risque-t-elle pas de partir en boucle infinie active si on ne démarre jamais MSN ?
7. Utiliser delete[] au lieu de delete...

[rdpdo]

MErci beaucoup beaucoup beaucoup j'ai résolu mon problème grace à tes commentaires

Super ! a +