[Mesa-dev] [PATCH 1/3] vulkan/wsi/x11: Better handle wsi_x11_connection_create failure

Dave Airlie airlied at gmail.com
Wed Nov 2 01:00:01 UTC 2016


For all 3,

Reviewed-by: Dave Airlie <airlied at redhat.com>

On 2 November 2016 at 10:56, Jason Ekstrand <jason at jlekstrand.net> wrote:
> Without this fix, the function would still end up returning NULL but it
> would put that NULL connection in the hash table which would be bad.
>
> Signed-off-by: Jason Ekstrand <jason at jlekstrand.net>
> Cc: "13.0" <mesa-stable at lists.freedesktop.org>
> Cc: Dave Airlie <airlied at redhat.com>
> ---
>  src/vulkan/wsi/wsi_common_x11.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan/wsi/wsi_common_x11.c
> index 8142847..eee2359 100644
> --- a/src/vulkan/wsi/wsi_common_x11.c
> +++ b/src/vulkan/wsi/wsi_common_x11.c
> @@ -118,6 +118,8 @@ wsi_x11_get_connection(struct wsi_device *wsi_dev,
>
>        struct wsi_x11_connection *wsi_conn =
>           wsi_x11_connection_create(alloc, conn);
> +      if (!wsi_conn)
> +         return NULL;
>
>        pthread_mutex_lock(&wsi->mutex);
>
> --
> 2.5.0.400.gff86faf
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list