[prev in list] [next in list] [prev in thread] [next in thread]
List: wine-devel
Subject: Re: [PATCH 1/2] shell32: Only call HCR_GetFolderAttributes with simple pidls
From: Fabian Maurer <dark.shadow4 () web ! de>
Date: 2022-01-31 22:25:39
Message-ID: 1805511.tdWV9SEqCh () arch
[Download RAW message or body]
Hello,
Any news on this patch series?
Regards,
Fabian Maurer
> The function returns false with not-simple pidls anyways.
> This fixes an "ERR" message when opening a control panel applet from
> explorer
>
> Signed-off-by: Fabian Maurer <dark.shadow4@web.de>
> ---
> dlls/shell32/shlfolder.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/shell32/shlfolder.c b/dlls/shell32/shlfolder.c
> index 10aafc160c0..8e3b561cfd1 100644
> --- a/dlls/shell32/shlfolder.c
> +++ b/dlls/shell32/shlfolder.c
> @@ -412,7 +412,7 @@ HRESULT SHELL32_GetItemAttributes (IShellFolder2 *psf,
> LPCITEMIDLIST pidl, LPDWO if (_ILIsDrive (pidl)) {
> *pdwAttributes &=
> SFGAO_HASSUBFOLDER|SFGAO_FILESYSTEM|SFGAO_FOLDER|SFGAO_FILESYSANCESTOR|
> SFGAO_DROPTARGET|SFGAO_HASPROPSHEET|SFGAO_CANLINK;
> - } else if (has_guid && HCR_GetFolderAttributes(pidl, &dwAttributes)) {
> + } else if (has_guid && _ILIsPidlSimple(pidl) &&
> HCR_GetFolderAttributes(pidl, &dwAttributes)) { *pdwAttributes =
> dwAttributes;
> } else if (_ILGetDataPointer (pidl)) {
> DWORD file_attr = _ILGetFileAttributes (pidl, NULL, 0);
> --
> 2.34.1
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic