[PATCH 2/2] shell32/iconcache: Break as soon as one shortcut failed to overlay

Gabriel Ivăncescu gabrielopcode at gmail.com
Mon Nov 26 04:22:04 CST 2018


There is no need to initialize and destroy shortcut icons that we haven't
even processed.

Signed-off-by: Gabriel Ivăncescu <gabrielopcode at gmail.com>
---
 dlls/shell32/iconcache.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/dlls/shell32/iconcache.c b/dlls/shell32/iconcache.c
index 44422ab..b9264bb 100644
--- a/dlls/shell32/iconcache.c
+++ b/dlls/shell32/iconcache.c
@@ -361,7 +361,6 @@ static BOOL get_imagelist_icon_size(int list, SIZE *size)
 static INT SIC_LoadIcon (const WCHAR *sourcefile, INT index, DWORD flags)
 {
     HICON hicons[ARRAY_SIZE(shell_imagelists)] = { 0 };
-    HICON hshortcuts[ARRAY_SIZE(hicons)] = { 0 };
     unsigned int i;
     SIZE size;
     INT ret = -1;
@@ -376,6 +375,7 @@ static INT SIC_LoadIcon (const WCHAR *sourcefile, INT index, DWORD flags)
 
     if (flags & GIL_FORSHORTCUT)
     {
+        HICON hshortcuts[ARRAY_SIZE(hicons)];
         BOOL failed = FALSE;
 
         for (i = 0; i < ARRAY_SIZE(hshortcuts); i++)
@@ -384,13 +384,15 @@ static INT SIC_LoadIcon (const WCHAR *sourcefile, INT index, DWORD flags)
             {
                 WARN("Failed to create shortcut overlaid icons.\n");
                 failed = TRUE;
+                break;
             }
         }
 
         if (failed)
         {
-            for (i = 0; i < ARRAY_SIZE(hshortcuts); i++)
-                DestroyIcon(hshortcuts[i]);
+            unsigned int k;
+            for (k = 0; k < i; k++)
+                DestroyIcon(hshortcuts[k]);
             flags &= ~GIL_FORSHORTCUT;
         }
         else
-- 
2.19.1




More information about the wine-devel mailing list