[PATCH vkd3d v2 2/3] vkd3d: Return E_INVALIDARG if a descriptor range overflows.

Conor McCarthy cmccarthy at codeweavers.com
Mon Aug 2 09:43:17 CDT 2021


Signed-off-by: Conor McCarthy <cmccarthy at codeweavers.com>
---
 libs/vkd3d/state.c | 3 +++
 tests/d3d12.c      | 3 ---
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/libs/vkd3d/state.c b/libs/vkd3d/state.c
index 588036d2..77383b25 100644
--- a/libs/vkd3d/state.c
+++ b/libs/vkd3d/state.c
@@ -613,6 +613,9 @@ static HRESULT d3d12_root_signature_init_root_descriptor_tables(struct d3d12_roo
             if (range->OffsetInDescriptorsFromTableStart != D3D12_DESCRIPTOR_RANGE_OFFSET_APPEND)
                 offset = range->OffsetInDescriptorsFromTableStart;
 
+            if (!vkd3d_bound_range(offset, range->NumDescriptors, UINT_MAX))
+                return E_INVALIDARG;
+
             table->ranges[j].offset = offset;
             table->ranges[j].descriptor_count = range->NumDescriptors;
             table->ranges[j].descriptor_magic = vkd3d_descriptor_magic_from_d3d12(range->RangeType);
diff --git a/tests/d3d12.c b/tests/d3d12.c
index 2f3fb63a..b9f8c46c 100644
--- a/tests/d3d12.c
+++ b/tests/d3d12.c
@@ -2870,10 +2870,7 @@ static void test_root_signature_limits(void)
     descriptor_ranges[0].NumDescriptors = 0x1001;
     descriptor_ranges[0].OffsetInDescriptorsFromTableStart = 0xfffff000;
     hr = create_root_signature(device, &root_signature_desc, &root_signature);
-    todo
     ok(hr == E_INVALIDARG, "Got unexpected hr %#x.\n", hr);
-    if (SUCCEEDED(hr))
-        ID3D12RootSignature_Release(root_signature);
 
     refcount = ID3D12Device_Release(device);
     ok(!refcount, "ID3D12Device has %u references left.\n", (unsigned int)refcount);
-- 
2.32.0




More information about the wine-devel mailing list