commit 787ccc8032f986477b69a340d877ff0a80ffa5b3
author Peter Dons Tychsen <donpedro@dhcppc4> Sat Jan 05 00:05:34 2008 +0100
committer Alexandre Julliard <julliard@winehq.org> Mon Jan 07 14:26:25 2008 +0100
tree cc1b02501b497ca6d1a65c6f25c8bacefbb0de03
parent 53f64a09cf696edcc6c346bab2a46585c6acabe5

devenum: Fix the implementation of IEnumMoniker::Skip(), to match the MSDN specs.

dlls/devenum/mediacatenum.c

diff --git a/dlls/devenum/mediacatenum.c b/dlls/devenum/mediacatenum.c
index d230ce1..331130b 100644
--- a/dlls/devenum/mediacatenum.c
+++ b/dlls/devenum/mediacatenum.c
@@ -800,9 +800,21 @@
 static HRESULT WINAPI DEVENUM_IEnumMoniker_Skip(LPENUMMONIKER iface, ULONG celt)
 {
     EnumMonikerImpl *This = (EnumMonikerImpl *)iface;
+    DWORD subKeys;
 
     TRACE("(%p)->(%d)\n", iface, celt);
 
+    /* Before incrementing, check if there are any more values to run thru.
+       Some programs use the Skip() function to get the amount of devices */
+    if(RegQueryInfoKeyW(This->hkey, NULL, NULL, NULL, &subKeys, NULL, NULL, NULL, NULL, NULL, NULL, NULL) != ERROR_SUCCESS)
+    {
+        return S_FALSE;
+    }
+    if((This->index + celt) >= subKeys)
+    {
+        return S_FALSE;
+    }
+
     This->index += celt;
 
     return S_OK;