Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>
On 05/10/2018 01:13, Dave Airlie wrote:
From: Dave Airlie <airl...@redhat.com>
If we fail the init path then don't call the free path.
Found by coverity
---
src/vulkan/wsi/wsi_common_wayland.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/vulkan/wsi/wsi_common_wayland.c
b/src/vulkan/wsi/wsi_common_wayland.c
index 6b34e21bd98..ce20583315a 100644
--- a/src/vulkan/wsi/wsi_common_wayland.c
+++ b/src/vulkan/wsi/wsi_common_wayland.c
@@ -455,8 +455,8 @@ wsi_wl_get_presentation_support(struct wsi_device
*wsi_device,
(struct wsi_wayland *)wsi_device->wsi[VK_ICD_WSI_PLATFORM_WAYLAND];
struct wsi_wl_display display;
- int ret = wsi_wl_display_init(wsi, &display, wl_display, false);
- wsi_wl_display_finish(&display);
+ if (wsi_wl_display_init(wsi, &display, wl_display, false))
+ wsi_wl_display_finish(&display);
return ret == 0;
}
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
