[PATCH vkd3d 3/5] libs/vkd3d: Validate signal_event_pfn.

Józef Kucia joseph.kucia at gmail.com
Fri Jan 19 10:17:23 CST 2018


From: Józef Kucia <jkucia at codeweavers.com>

Signed-off-by: Józef Kucia <jkucia at codeweavers.com>
---
 libs/vkd3d/device.c |  5 +++++
 tests/vkd3d_api.c   | 16 +++++++++-------
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/libs/vkd3d/device.c b/libs/vkd3d/device.c
index af3f0534def9..7a9951122e4d 100644
--- a/libs/vkd3d/device.c
+++ b/libs/vkd3d/device.c
@@ -221,6 +221,11 @@ static HRESULT vkd3d_instance_init(struct vkd3d_instance *instance,
     VkResult vr;
     HRESULT hr;
 
+    if (!create_info->signal_event_pfn)
+    {
+        ERR("Invalid signal event function pointer.\n");
+        return E_INVALIDARG;
+    }
     if (!create_info->create_thread_pfn != !create_info->join_thread_pfn)
     {
         ERR("Invalid create/join thread function pointers.\n");
diff --git a/tests/vkd3d_api.c b/tests/vkd3d_api.c
index 41b09fbcc0fa..c387abecc196 100644
--- a/tests/vkd3d_api.c
+++ b/tests/vkd3d_api.c
@@ -74,9 +74,7 @@ static void test_create_instance(void)
     ULONG refcount;
     HRESULT hr;
 
-    memset(&create_info, 0, sizeof(create_info));
-    create_info.wchar_size = sizeof(WCHAR);
-
+    create_info = instance_default_create_info;
     hr = vkd3d_create_instance(&create_info, &instance);
     ok(hr == S_OK, "Failed to create instance, hr %#x.\n", hr);
     refcount = vkd3d_instance_incref(instance);
@@ -85,11 +83,15 @@ static void test_create_instance(void)
     refcount = vkd3d_instance_decref(instance);
     ok(!refcount, "Instance has %u references left.\n", refcount);
 
-    create_info.signal_event_pfn = signal_event;
+    create_info = instance_default_create_info;
+    create_info.wchar_size = 1;
     hr = vkd3d_create_instance(&create_info, &instance);
-    ok(hr == S_OK, "Failed to create instance, hr %#x.\n", hr);
-    refcount = vkd3d_instance_decref(instance);
-    ok(!refcount, "Instance has %u references left.\n", refcount);
+    ok(hr == E_INVALIDARG, "Got unexpected hr %#x.\n", hr);
+
+    create_info = instance_default_create_info;
+    create_info.signal_event_pfn = NULL;
+    hr = vkd3d_create_instance(&create_info, &instance);
+    ok(hr == E_INVALIDARG, "Got unexpected hr %#x.\n", hr);
 }
 
 static void test_create_device(void)
-- 
2.13.6




More information about the wine-devel mailing list