Problème avec GetOpenFileName()Résolu

Question

Salut,
J'ai un probl&#232;me avec la fonction GetOpenFileName()

BOOL CALLBACK ExplorerDlgProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    HWND Path;

    switch(message)
    {
                case IDC_BTNUPLOAD:
                {
                    OPENFILENAME file;
                    char FileName[MAX_PATH];
                    char Pathname[MAX_PATH];
                    
                    ZeroMemory(FileName, sizeof(FileName));
                    ZeroMemory(&file, sizeof(file));

                    file.hwndOwner = hwnd;
                    file.hInstance = NULL;
                    file.lpstrCustomFilter = NULL;
                    file.nMaxCustFilter = 1;
                    file.nFilterIndex = 1;
                    file.lpstrFileTitle = NULL;
                    file.nMaxFileTitle = NULL;
                    file.lpstrInitialDir = NULL;
                    file.lpstrDefExt = NULL;
                    file.lCustData = NULL;
                    file.lpfnHook = NULL;
                    file.lpTemplateName = NULL;
                    file.lpstrFilter = "Tous les Fichier (*.*)\0*.*\0";
                    file.lpstrDefExt = "*.*";

                    file.nMaxFile = sizeof(FileName);
                    file.lpstrFile = FileName;

                    file.Flags = OFN_EXPLORER|OFN_FILEMUSTEXIST|OFN_HIDEREADONLY;
                    file.lStructSize = sizeof(file);

                    if (GetOpenFileName(&file))
                    {
                        GetWindowText(GetDlgItem(hwnd, IDC_LISTE_PATH), Pathname, 512);
                        UploadFile(FileName, Pathname);
                    }
                }
              break;
    }    

    return 0;
}

Voil&#224;, parfois pendant que la fen&#234;tre est ouverte j'obtient un plantage de l'application (avant de cliquer sur OK).

Exception non g&#233;r&#233;e &#224; 0x7ca614ae dans Client trojan v2.exe:0xC0000005: Violation d'acc&#232;s lors de la lecture de l'emplacement 0x01384f90

C'est tr&#232;s bizarre, car &#231;a se passe de fa&#231;on al&#233;atoire on dirait. Souvent c'est apr&#232;s un deuxi&#232;me appel de la fonction que &#231;a plante. Je n'ai pas encore une grande connaissance du C alors je me demandais si vous auriez une explication, ou peut-&#234;tre qu'une chose pareil vous serait d&#233;j&#224; arriv&#233; (l'exp&#233;rience quoi). J'ai regard&#233; partout dans l'appli je n'ai pas trouver de m&#233;moire qui semble empi&#233;ter.

Aussi je voudrais une info, la fonction GetOpenFileName() &#233;tant autoblocante, comment ce fait-il que je peux continuer &#224; utiliser mon programme (y compris la fen&#234;tre qui a lanc&#233; le get) sans qu'il bloque ? Et m&#234;me appeler une deuxi&#232;me fen&#234;tre d'ouverture (en mettant hwndOwner = 0)

A+

http://vbaddons.free.fr

if (CPossible)

BruNews · Accepted Answer

lpstrFilter - est lu en tant que chaine 'multi part' o&#249; chaque partie est normalement termin&#233;e par un z&#233;ro et le dernier &#233;lem DOIT terminer par 2 z&#233;ros, exemple "*.*\0*.*\0\0".
Vire les ZeroMemory inutiles: Filename[0] = 0; suffit pour finir une chaine.
file.lStructSize = sizeof(OPENFILENAME); !!!

GetWindowText(GetDlgItem(hwnd, IDC_LISTE_PATH), Pathname, 512);
d'o&#249; sort ce 512 ???

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

cs_max12 · Answer

En passant, je sais que mettre &#224; null &#233;tait inutile &#224; cause du ZeroMemory() mais j'&#233;tais rendu pas mal d&#233;sesp&#233;r&#233; :P

http://vbaddons.free.fr

if (CPossible)

SAKingdom · Answer

Oui pourquoi 512. La limite de Pathname est MAX_PATH qui a une valeur de 255 ou 256 ch&#233; plus trop. &#199;a serais plus prudent de faire GetWindowText(GetDlgItem(hwnd, IDC_LISTE_PATH), Pathname, MAX_PATH ou sizeof(Pathname));

___________________________________________
Les plus grands esprits trouvent toujours une solution

cs_max12 · Answer

Pour le GetWindowText() je m'en suis aper&#231;u apr&#232;s le post, je venais tout juste de changer le code cette section et j'avais oublier les nouvelles dimentions. Merci pour les conseils, mais &#231;a ne r&#232;gle pas, je vais continuer &#224; chercher d'&#233;ventuel d&#233;passement dans le code, mais la au moins je sais qu'elle est maintenant initialis&#233;e comme il faut.

A+

http://vbaddons.free.fr

if (CPossible)

SAKingdom · Answer

T'as peut-etre pas besoin de mettre autant d'info dans ton OPENFILENAME
Tien ceci peut faire l'affaire.

FileName[0] = '\0';
ZeroMemory(&file, sizeof(file));
    
file.lStructSize   = sizeof(file);
file.hwndOwner     = hwnd;
file.lpstrFilter   = "Tous les fichiers (*.*) \0 *.* \0\0";
file.lpstrFile     = FileName;
file.nMaxFile      = MAX_PATH;
file.lpstrDefExt   = "*.*";
file.lpstrTitle    = "Ouvrir un fichier...";
file.Flags         = OFN_EXPLORER|OFN_FILEMUSTEXIST|OFN_HIDEREADONLY;

___________________________________________
Les plus grands esprits trouvent toujours une solution

Problème avec GetOpenFileName()

5 réponses

Votre réponse

Discussions similaires