vulkaninfo: Add checks for some struct allocation

KhronosGroup / Vulkan-LoaderAndValidationLayers

**Deprecated repository** for Vulkan loader and validation layers

Apache License 2.0

414 stars 172 forks source link

vulkaninfo: Add checks for some struct allocation #2542

Closed jeremyk-lunarg closed 6 years ago

jeremyk-lunarg commented 6 years ago

Added checks after attempted memory allocation in AppGpuInit and before memory frees in AppGpuDestroy to fix crashing.

mikew-lunarg commented 6 years ago

FWIW attempting to test this PR w Devsim from VulkanTools, but getting bogged down in updating the LVL submodules. Will report ASAP.

mikew-lunarg commented 6 years ago

Verified that this fixes the crash that inspired this thread 👍

jeremyk-lunarg commented 6 years ago

Since I have addressed every suggestion so far, the PR is passing all CI testing, and @mikew-lunarg needs this for his project, if there are no further comments on this by 11AM tomorrow (April 4) I will merge the PR.