=?UTF-8?Q?J=C3=B3zef=20Kucia=20?=: vkd3d: Set pointer to NULL if heap creation fails.

Alexandre Julliard julliard at winehq.org
Wed Sep 12 16:37:56 CDT 2018


Module: vkd3d
Branch: master
Commit: 605cb12eca2cdfa9c9b4ed29668060d0cf9c1d0a
URL:    https://source.winehq.org/git/vkd3d.git/?a=commit;h=605cb12eca2cdfa9c9b4ed29668060d0cf9c1d0a

Author: Józef Kucia <jkucia at codeweavers.com>
Date:   Wed Sep 12 15:19:58 2018 +0200

vkd3d: Set pointer to NULL if heap creation fails.

Signed-off-by: Józef Kucia <jkucia at codeweavers.com>
Signed-off-by: Henri Verbeet <hverbeet at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 libs/vkd3d/device.c | 3 +++
 tests/d3d12.c       | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/libs/vkd3d/device.c b/libs/vkd3d/device.c
index e8bf7c9..d39eef2 100644
--- a/libs/vkd3d/device.c
+++ b/libs/vkd3d/device.c
@@ -1946,7 +1946,10 @@ static HRESULT STDMETHODCALLTYPE d3d12_device_CreateHeap(ID3D12Device *iface,
             iface, desc, debugstr_guid(iid), heap);
 
     if (FAILED(hr = d3d12_heap_create(device, desc, &object)))
+    {
+        *heap = NULL;
         return hr;
+    }
 
     return return_interface(&object->ID3D12Heap_iface, &IID_ID3D12Heap, iid, heap);
 }
diff --git a/tests/d3d12.c b/tests/d3d12.c
index 5a39f0c..221157f 100644
--- a/tests/d3d12.c
+++ b/tests/d3d12.c
@@ -1854,9 +1854,12 @@ static void test_create_heap(void)
     desc.Flags = D3D12_HEAP_FLAG_ALLOW_ONLY_NON_RT_DS_TEXTURES | D3D12_HEAP_FLAG_ALLOW_DISPLAY;
     hr = ID3D12Device_CreateHeap(device, &desc, &IID_ID3D12Heap, (void **)&heap);
     ok(hr == E_INVALIDARG, "Got unexpected hr %#x.\n", hr);
+
+    heap = (void *)0xdeadbeef;
     desc.Flags = D3D12_HEAP_FLAG_ALLOW_ONLY_RT_DS_TEXTURES | D3D12_HEAP_FLAG_ALLOW_DISPLAY;
     hr = ID3D12Device_CreateHeap(device, &desc, &IID_ID3D12Heap, (void **)&heap);
     ok(hr == E_INVALIDARG, "Got unexpected hr %#x.\n", hr);
+    ok(!heap, "Got unexpected pointer %p.\n", heap);
 
     for (i = 0; i < ARRAY_SIZE(tests); ++i)
     {




More information about the wine-cvs mailing list