Alistair Leslie-Hughes : dpnet: Handle unknown service GUID in enum_services_providers.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 83cce966f4cc4692035b18bf0766caa630702b28
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=83cce966f4cc4692035b18bf0766caa630702b28

Author: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
Date:   Fri Feb 24 06:45:29 2017 +0000

dpnet: Handle unknown service GUID in enum_services_providers.

Signed-off-by: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/dpnet/peer.c         |  4 ++--
 dlls/dpnet/tests/client.c | 12 ++++++++++++
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/dlls/dpnet/peer.c b/dlls/dpnet/peer.c
index 91a7e3e..e8a60c3 100644
--- a/dlls/dpnet/peer.c
+++ b/dlls/dpnet/peer.c
@@ -147,10 +147,10 @@ HRESULT enum_services_providers(const GUID * const service, DPN_SERVICE_PROVIDER
     if(!returned || !buf_size)
         return E_POINTER;
 
-    *returned = 0;
-
     if(!service)
     {
+        *returned = 0;
+
         res = RegOpenKeyExW(HKEY_LOCAL_MACHINE, serviceproviders, 0, KEY_READ, &key);
         if(res == ERROR_FILE_NOT_FOUND)
             return DPNERR_DOESNOTEXIST;
diff --git a/dlls/dpnet/tests/client.c b/dlls/dpnet/tests/client.c
index ee2722b..6503bc3 100644
--- a/dlls/dpnet/tests/client.c
+++ b/dlls/dpnet/tests/client.c
@@ -211,6 +211,12 @@ static void test_enum_service_providers(void)
     }
 
     ok(HeapFree(GetProcessHeap(), 0, serv_prov_info), "Failed freeing server provider info\n");
+
+    /* Invalid GUID */
+    items = 88;
+    hr = IDirectPlay8Client_EnumServiceProviders(client, &appguid, NULL, serv_prov_info, &size, &items, 0);
+    ok(hr == DPNERR_DOESNOTEXIST, "IDirectPlay8Peer_EnumServiceProviders failed with %x\n", hr);
+    ok(items == 88, "Found adapter %d\n", items);
 }
 
 static void test_enum_hosts(void)
@@ -571,6 +577,12 @@ static void test_enum_service_providers_peer(void)
 
     serv_prov_info -= items; /* set pointer back */
     ok(HeapFree(GetProcessHeap(), 0, serv_prov_info), "Failed freeing server provider info\n");
+
+    /* Invalid GUID */
+    items = 88;
+    hr = IDirectPlay8Peer_EnumServiceProviders(peer, &appguid, NULL, serv_prov_info, &size, &items, 0);
+    ok(hr == DPNERR_DOESNOTEXIST, "IDirectPlay8Peer_EnumServiceProviders failed with %x\n", hr);
+    ok(items == 88, "Found adapter %d\n", items);
 }
 
 static void test_enum_hosts_peer(void)