Seems reasonable

Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net>

On Thu, Apr 6, 2017 at 8:56 PM, Ben Widawsky <b...@bwidawsk.net> wrote:

> The API/entry point in mesa already checks the correct behavior,
> however, it's possible to be handled by another implementation and those
> implementations should not be able to abuse a weird combination of count
> and pointer.
>
> This fixes CID 1403193
>
> Cc: Mark Janes <mark.a.ja...@intel.com>
> Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
> ---
>  src/gbm/backends/dri/gbm_dri.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/src/gbm/backends/dri/gbm_dri.c b/src/gbm/backends/dri/gbm_
> dri.c
> index 8df6a6b64d..71590d7110 100644
> --- a/src/gbm/backends/dri/gbm_dri.c
> +++ b/src/gbm/backends/dri/gbm_dri.c
> @@ -1266,6 +1266,9 @@ gbm_dri_surface_create(struct gbm_device *gbm,
>        return NULL;
>     }
>
> +   if (count)
> +      assert(modifiers);
> +
>     /* It's acceptable to create an image with INVALID modifier in the
> list,
>      * but it cannot be on the only modifier (since it will certainly fail
>      * later). While we could easily catch this after modifier creation,
> doing
> --
> 2.12.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to