[PATCH v4 1/1] shell32: Try to lookup icon, with fallback on failure

[James McDonnell]

From: James McDonnell

Try to lookup icon information from shell32.dll. If an icon is not
found, default to the help icon.

This resolves the issue where in affinity photos installer it
would crash because hIcon was returned as 0
---
 dlls/shell32/iconcache.c | 30 +++++++++++++++++++++++-------
 1 file changed, 23 insertions(+), 7 deletions(-)

diff --git a/dlls/shell32/iconcache.c b/dlls/shell32/iconcache.c
index 8f9519d2ca8..edcaf1a576e 100644
--- a/dlls/shell32/iconcache.c
+++ b/dlls/shell32/iconcache.c
@@ -976,6 +976,16 @@ INT WINAPI SHGetIconOverlayIndexW(LPCWSTR pszIconPath, INT iIconIndex)
   return -1;
 }

+/****************************************************************************
+ * StockIconIdToResourceId    [SHELL32.@]
+ *
+ * Map the SHSTOCKICONID value to shresdef.h IDI_ values
+ */
+INT StockIconIdToResourceId(SHSTOCKICONID id)
+{
+  return id + 1;
+}
+
 /****************************************************************************
  * SHGetStockIconInfo [SHELL32.@]
  *
@@ -1000,19 +1010,25 @@ HRESULT WINAPI SHGetStockIconInfo(SHSTOCKICONID id, UINT flags, SHSTOCKICONINFO

GetSystemDirectoryW(sii->szPath, MAX_PATH);

-    /* no icons defined: use default */
-    sii->iIcon = -IDI_SHELL_FILE;
+    sii->iIcon = id;
     lstrcatW(sii->szPath, L"\shell32.dll");

-    if (flags)
-        FIXME("flags 0x%x not implemented\n", flags);
+    if (flags & ~SHGSI_ICON)
+        FIXME("unhandled flags 0x%x\n", flags);

sii->hIcon = NULL;
-    if (flags & SHGSI_ICON)
-        sii->hIcon = LoadIconW(GetModuleHandleW(sii->szPath), MAKEINTRESOURCEW(sii->iIcon));
+    if (flags & SHGSI_ICON) {
+        sii->hIcon = LoadIconW(GetModuleHandleW(sii->szPath), MAKEINTRESOURCEW(StockIconIdToResourceId(id)));
+
+        // many icons do not exist yet in wine, map to the question mark
+        if (!sii->hIcon) {
+          sii->iIcon = IDI_SHELL_HELP - 1;
+          sii->hIcon = LoadIconW(GetModuleHandleW(sii->szPath), MAKEINTRESOURCEW(IDI_SHELL_HELP));
+        }
+    }
     sii->iSysImageIndex = -1;

-    TRACE("%3d: returning %s (%d)\n", id, debugstr_w(sii->szPath), sii->iIcon);
+    TRACE("%3d: returning %s (%d), icon handle=%p\n", id, debugstr_w(sii->szPath), sii->iIcon, sii->hIcon);

return S_OK;
 }