commit c14c2ebfb70ae9e6d0d807cdb93ec2a025217278
author Nikolay Sivov <nsivov@codeweavers.com> Wed Sep 27 11:13:45 2017 +0300
committer Alexandre Julliard <julliard@winehq.org> Wed Sep 27 10:20:15 2017 +0200
tree af5e1df9142ad5ef7d5dbb82da090255f7a83dfe
parent d484650b6d95b22000744f9316564a97bfb0a379

shell32: Fix IEnumIDList::Next() called for multiple items.

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

dlls/shell32/shelldispatch.c

diff --git a/dlls/shell32/shelldispatch.c b/dlls/shell32/shelldispatch.c
index 3763ff8..148c45f 100644
--- a/dlls/shell32/shelldispatch.c
+++ b/dlls/shell32/shelldispatch.c
@@ -1284,7 +1284,10 @@
 
     if (This->item_count)
     {
-        LPITEMIDLIST *pidls = HeapAlloc(GetProcessHeap(), 0, This->item_count * sizeof(*pidls));
+        LPITEMIDLIST *pidls;
+        ULONG fetched;
+
+        pidls = HeapAlloc(GetProcessHeap(), 0, This->item_count * sizeof(*pidls));
         This->item_names = HeapAlloc(GetProcessHeap(), 0, This->item_count * sizeof(*This->item_names));
 
         if (!pidls || !This->item_names)
@@ -1296,7 +1299,7 @@
         }
 
         IEnumIDList_Reset(enumidlist);
-        if (IEnumIDList_Next(enumidlist, This->item_count, pidls, NULL) == S_OK)
+        if (IEnumIDList_Next(enumidlist, This->item_count, pidls, &fetched) == S_OK)
             idlist_sort(pidls, 0, This->item_count - 1, folder->folder);
 
         for (i = 0; i < This->item_count; i++)