[Mesa-dev] [PATCH] vulkan/wsi/x11: don't crash on null visual

2016-12-22 Thread Arda Coskunses
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. --- src/vulkan/wsi/wsi_common_x11.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/vulkan/wsi/wsi_common_x11.c

Re: [Mesa-dev] [PATCH] vulkan/wsi/x11: don't crash on null visual

2016-12-22 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand On Thu, Dec 22, 2016 at 1:55 PM, Arda Coskunses wrote: > 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. > --- > src/vulkan/wsi/wsi_c

Re: [Mesa-dev] [PATCH] vulkan/wsi/x11: don't crash on null visual

2016-12-22 Thread Jason Ekstrand
pushed On Thu, Dec 22, 2016 at 1:56 PM, Jason Ekstrand wrote: > Reviewed-by: Jason Ekstrand > > On Thu, Dec 22, 2016 at 1:55 PM, Arda Coskunses > wrote: > >> When application window closed unexpectedly due to >> lost window visualtypes getting invlaid parameters >> which is causing a crash. Ne

Re: [Mesa-dev] [PATCH] vulkan/wsi/x11: don't crash on null visual

2016-12-23 Thread Eero Tamminen
Hi, Either this, or the other commit fixed segfault with vkreplay on DOTA2 Vulkan trace (segfault happened in trace before DOTA2 came to its main menu). - Eero On 23.12.2016 00:10, Jason Ekstrand wrote: pushed On Thu, Dec 22, 2016 at 1:56 PM, Jason Ekstrand mailto:ja...@jlekstrand.