commit 01dd363e671ac3a457bc3ff60b43b04e108c33fc
author Arda Coskunses <acoskunses@gmail.com> Thu Dec 22 14:55:03 2016 -0700
committer Jason Ekstrand <jason.ekstrand@intel.com> Thu Dec 22 14:09:34 2016 -0800
tree c76ae74b2ca2c78dd9022a402fdd2fdb7749293b
parent 7a4ea95f1c5257e79a7a207eccff81c558c04d46

vulkan/wsi/x11: don't crash on null visual

When application window closed unexpectedly due to
lost window visualtypes getting invlaid parameters
which is causing a crash. Necessary check is added
to prevent the crash.

Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Cc: "13.0" <mesa-stable@lists.freedesktop.org>

src/vulkan/wsi/wsi_common_x11.c

diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan/wsi/wsi_common_x11.c
index 25ba0c1..037aa50 100644
--- a/src/vulkan/wsi/wsi_common_x11.c
+++ b/src/vulkan/wsi/wsi_common_x11.c
@@ -349,6 +349,9 @@
    xcb_visualtype_t *visual =
       get_visualtype_for_window(conn, window, &visual_depth);
 
+   if (!visual)
+      return VK_ERROR_SURFACE_LOST_KHR;
+
    geom = xcb_get_geometry_reply(conn, geom_cookie, &err);
    if (geom) {
       VkExtent2D extent = { geom->width, geom->height };