vulkano-rs / vulkano

Safe and rich Rust wrapper around the Vulkan API
Apache License 2.0
4.45k stars 435 forks source link

fix null handling of array fields in autogen properties #2518

Closed 0xcaff closed 4 months ago

0xcaff commented 4 months ago

Overview

Upon closer inspection of VkPhysicalDeviceHostImageCopyPropertiesEXT it appears the nullability of the pCopySrcLayouts and pCopyDstLayouts was not handled correctly.

These changes ensure that if an array field has a null value, its value is set to None instead of trying to dereference the pointer. It does not yet do one request for the size and a second for the values as outlined in the docs. This will be done in a follow up PR.

Original Bug Report: https://discord.com/channels/937149253296476201/937153798982479912/1230477146062061629

vulkaninfo from crashing device vulkaninfo.txt

Changelog

### Bugs fixed
- fix null handling of array fields in autogen properties