[PATCH] dpnet: Handle unknown service GUID in enum_services_providers

Alistair Leslie-Hughes leslie_alistair at hotmail.com
Fri Feb 24 00:45:29 CST 2017


Signed-off-by: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
---
 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)
-- 
1.9.1




More information about the wine-patches mailing list