Discussion :

[melleb]

Bonjour,

j'ai un ptit souci à l'exécution de mon appli développée en c++ sous VS2005

Voilà ce que j'ai et d'où je soupçonne que vienne le pb :

dans le .h

Code :

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

1
2
3
4
5
6
7
8
class MaClasse
{
public:
    STDMETHOD (SetName) (LPWSTR Name);
 
protected:
    LPWSTR m_Name;
}

dans le .ccp

Code :

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

1
2
3
4
5
SDTMETHODIMP Maclasse::SetName(LPWSTR Name)
{
    m_Name = Name;
    return S_OK;
}

et pour l'utilisation :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
MaMathode(ClasseParam param)
{
    MaClasse toto;
    LPWSTR nom(NULL);
    size_t convertedChars = 0;
    HRESULT Result;
 
    NewsSTR(L"",&nom);
    mbstowcs_s(&convertedChars, nom, strlen(param.Nom) + 1, param.Nom, 18);    // param.Nom étant de type TCHAR[17]
    Result = toto.SetName(nom);
    CoTaskMemFree(nom);
}

Et donc, voilà
ça compile très bien,
mais à l'exécution, ça pète de n'importe où
jamais au même endroit...
(un truc du genre 'violation d'accès de la lecture...')

et donc, je soupçonne que ça vienne du mbstowcs_s, car si je le mets en commentaire, bah, ça va...

je suppose que je m'y prends mal,
mais comment dois-je faire??

Merci

[Mac LAK]

je suppose que je m'y prends mal,
mais comment dois-je faire??

Déjà, tu mets les mauvais paramètres :

1. Quelle est la taille mémoire allouée pour nom ? A priori, un seul wchar_t (le zéro terminal), vu le NewsSTR...
2. Tu passes strlen(param.Nom)+1 pour sizeInWords, alors que tu es censé passer la taille du buffer wcstr (nom pour toi).
3. Pour le paramètre count, tu mets 18 en dur au lieu de mettre la taille de ta chaîne MBCS source (param.Nom), ou _TRUNCATE...

A priori, c'est surtout la réservation de mémoire de nom qui te fait exploser à l'exécution. Mais les paramètres de mbstowcs_s ne sont pas cohérents quoi qu'il en soit.

[melleb]

A priori, c'est surtout la réservation de mémoire de nom qui te fait exploser à l'exécution. Mais les paramètres de mbstowcs_s ne sont pas cohérents quoi qu'il en soit.

Euh, oui, ok
mais comment je fais pour bien réserver la mémoire de 'nom'??

merci

[melleb]

J'ai fait ça :

Code :

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

nom = (LPWSTR) new wchar_t[strlen(param.Nom) + 1]

et j'ai utilisé _TRUNCATE

ça a l'air de marcher

[Médinoc]

Sauf que tu retires cet horrible cast.