[Mesa-dev] [PATCH mesa 2/2] wsi/wayland: only finish() a successfully init()ed display

[Philipp Zabel]

On Mon, 2018-11-05 at 10:00 +0000, Eric Engestrom wrote:
> Cc: Dave Airlie <airlied at gmail.com>
> Cc: Philipp Zabel <p.zabel at pengutronix.de>
> Signed-off-by: Eric Engestrom <eric.engestrom at intel.com>

Reviewed-by: Philipp Zabel <p.zabel at pengutronix.de>

> ---
>  src/vulkan/wsi/wsi_common_wayland.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/vulkan/wsi/wsi_common_wayland.c b/src/vulkan/wsi/wsi_common_wayland.c
> index a799b427621d72b2927c..5d79d377f12df81b06ba 100644
> --- a/src/vulkan/wsi/wsi_common_wayland.c
> +++ b/src/vulkan/wsi/wsi_common_wayland.c
> @@ -456,7 +456,8 @@ wsi_wl_get_presentation_support(struct wsi_device *wsi_device,
>  
>     struct wsi_wl_display display;
>     VkResult ret = wsi_wl_display_init(wsi, &display, wl_display, false);
> -   wsi_wl_display_finish(&display);
> +   if (ret == VK_SUCCESS)
> +      wsi_wl_display_finish(&display);
>  
>     return ret == VK_SUCCESS;
>  }

regards
Philipp