Re: [PATCH v8 0/6] MR1857: Draft: comdlg32: IFileDialog navigation bar

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/comdlg32/navbar.c:
> - expanded = HeapAlloc(GetProcessHeap(), 0, expanded_cch * sizeof(WCHAR));
> - if (!expanded)
> -
>    goto cleanup;
> -
> - expanded_cch = ExpandEnvironmentStringsW(text, expanded, expanded_cch);
> - if (!expanded_cch)
> -
>    goto cleanup;
> -
> - /* check if we have a valid path */
> - if (GetFileAttributesW(expanded) == INVALID_FILE_ATTRIBUTES)
> - {
> -
>    WCHAR spec[MAX_PATH + 128] = {0},
> -
>          caption[MAX_PATH + 128] = {0};
> -
>    LoadStringW(COMDLG32_hInstance, IDS_WINE_CANT_FIND_ADDRESS, spec, ARRAY_SIZE(spec));
> -
>    wsprintfW(caption, spec, text);
> -
>    MessageBoxW(info->pathedit_hwnd, caption, NULL, MB_ICONERROR | MB_OK);

According to msdn, the hwnd passed in here should not be a child window.