diff options
author | Karmjit Mahil <Karmjit.Mahil@imgtec.com> | 2023-05-04 16:37:28 +0100 |
---|---|---|
committer | Marge Bot <emma+marge@anholt.net> | 2023-05-16 18:09:03 +0000 |
commit | bac048e1dfed0ddfa7d95e55565d0ac486997b9b (patch) | |
tree | d00f65ab69842395d025552c8742805209c29cc3 /src/imagination/vulkan/pvr_descriptor_set.c | |
parent | 7cb77cb089cfa38c06dfaaed694af14178f70601 (diff) | |
download | mesa-bac048e1dfed0ddfa7d95e55565d0ac486997b9b.tar.gz |
pvr: Add missing NULL checks in some vkDestroy...() functions
Seems like `dEQP-VK.api.null_handle.destroy_device` was already
passing but let's add the null check in case of future changes
which might not accept NULL.
Fixes:
dEQP-VK.api.null_handle.destroy_descriptor_set_layout
dEQP-VK.api.null_handle.destroy_pipeline_layout
dEQP-VK.api.null_handle.destroy_query_pool
Signed-off-by: Karmjit Mahil <Karmjit.Mahil@imgtec.com>
Reviewed-by: Frank Binns <frank.binns@imgtec.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22918>
Diffstat (limited to 'src/imagination/vulkan/pvr_descriptor_set.c')
-rw-r--r-- | src/imagination/vulkan/pvr_descriptor_set.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/imagination/vulkan/pvr_descriptor_set.c b/src/imagination/vulkan/pvr_descriptor_set.c index 57b84c395af..1855876839c 100644 --- a/src/imagination/vulkan/pvr_descriptor_set.c +++ b/src/imagination/vulkan/pvr_descriptor_set.c @@ -675,6 +675,9 @@ void pvr_DestroyDescriptorSetLayout(VkDevice _device, PVR_FROM_HANDLE(pvr_descriptor_set_layout, layout, _set_layout); PVR_FROM_HANDLE(pvr_device, device, _device); + if (!layout) + return; + pvr_descriptor_set_layout_free(device, pAllocator, layout); } @@ -1031,6 +1034,9 @@ void pvr_DestroyPipelineLayout(VkDevice _device, PVR_FROM_HANDLE(pvr_device, device, _device); PVR_FROM_HANDLE(pvr_pipeline_layout, layout, _pipelineLayout); + if (!layout) + return; + vk_object_free(&device->vk, pAllocator, layout); } |