Only return FALSE in IEnumIDList_fnNext if no item could be fetched at

all.

Only return FALSE in IEnumIDList_fnNext if no item could be fetched at
ad1fbf2e · Martin Fuchs · Alexandre Julliard · 26904868 · ad1fbf2e
Commit ad1fbf2e authored Jul 22, 2003 by Martin Fuchs Committed by Alexandre Julliard Jul 22, 2003
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

enumidlist.c dlls/shell32/enumidlist.c +5 -2

No files found.
--- a/dlls/shell32/enumidlist.c
+++ b/dlls/shell32/enumidlist.c
@@ -443,11 +443,14 @@ static HRESULT WINAPI IEnumIDList_fnNext(
 	{ return E_INVALIDARG;
 	}

+	if(celt > 0 && !This->mpCurrent)
+	{ return S_FALSE;
+	}
+
 	for(i = 0; i < celt; i++)
 	{ if(!(This->mpCurrent))
-	  { hr =  S_FALSE;
 	    break;
-	  }
+
 	  temp = ILClone(This->mpCurrent->pidl);
 	  rgelt[i] = temp;
 	  This->mpCurrent = This->mpCurrent->pNext;