[PATCH vkd3d 2/3] vkd3d: Reject empty descriptor table ranges.
Conor McCarthy
cmccarthy at codeweavers.com
Mon Oct 11 09:33:20 CDT 2021
Signed-off-by: Conor McCarthy <cmccarthy at codeweavers.com>
---
libs/vkd3d/state.c | 6 ++++++
tests/d3d12.c | 3 ---
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/libs/vkd3d/state.c b/libs/vkd3d/state.c
index 6b088fff..92978a1b 100644
--- a/libs/vkd3d/state.c
+++ b/libs/vkd3d/state.c
@@ -345,6 +345,12 @@ static HRESULT d3d12_root_signature_info_count_descriptors(struct d3d12_root_sig
const D3D12_DESCRIPTOR_RANGE *range = &table->pDescriptorRanges[i];
unsigned int binding_count;
+ if (!range->NumDescriptors)
+ {
+ WARN("A descriptor register range is empty.\n");
+ return E_INVALIDARG;
+ }
+
if (unbounded && range->OffsetInDescriptorsFromTableStart == D3D12_DESCRIPTOR_RANGE_OFFSET_APPEND)
{
WARN("An unbounded range with offset D3D12_DESCRIPTOR_RANGE_OFFSET_APPEND occurs after "
diff --git a/tests/d3d12.c b/tests/d3d12.c
index 8ae40afa..775fabff 100644
--- a/tests/d3d12.c
+++ b/tests/d3d12.c
@@ -2707,10 +2707,7 @@ static void test_create_root_signature(void)
descriptor_ranges[0].NumDescriptors = 0;
root_parameters[0].DescriptorTable.NumDescriptorRanges = 1;
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);
/* descriptor register range overflow
* Windows results vary for overflowing to zero, but anything beyond that is invalid. */
--
2.32.0
More information about the wine-devel
mailing list