<div dir="ltr">Thanks to both of you.  Pushed.<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 2, 2016 at 3:11 AM, Eric Engestrom <span dir="ltr"><<a href="mailto:eric.engestrom@imgtec.com" target="_blank">eric.engestrom@imgtec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tuesday, 2016-11-01 17:56:05 -0700, Jason Ekstrand wrote:<br>
> Without this fix, the function would still end up returning NULL but it<br>
> would put that NULL connection in the hash table which would be bad.<br>
><br>
> Signed-off-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br>
> Cc: "13.0" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
> Cc: Dave Airlie <<a href="mailto:airlied@redhat.com">airlied@redhat.com</a>><br>
<br>
</span>Series is:<br>
Reviewed-by: Eric Engestrom <<a href="mailto:eric.engestrom@imgtec.com">eric.engestrom@imgtec.com</a>><br>
<div class="HOEnZb"><div class="h5"><br>
> ---<br>
>  src/vulkan/wsi/wsi_common_x11.<wbr>c | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
><br>
> diff --git a/src/vulkan/wsi/wsi_common_<wbr>x11.c b/src/vulkan/wsi/wsi_common_<wbr>x11.c<br>
> index 8142847..eee2359 100644<br>
> --- a/src/vulkan/wsi/wsi_common_<wbr>x11.c<br>
> +++ b/src/vulkan/wsi/wsi_common_<wbr>x11.c<br>
> @@ -118,6 +118,8 @@ wsi_x11_get_connection(struct wsi_device *wsi_dev,<br>
><br>
>        struct wsi_x11_connection *wsi_conn =<br>
>           wsi_x11_connection_create(<wbr>alloc, conn);<br>
> +      if (!wsi_conn)<br>
> +         return NULL;<br>
><br>
>        pthread_mutex_lock(&wsi-><wbr>mutex);<br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</div></div></blockquote></div><br></div></div></div>