Seems reasonable Reviewed-by: Jason Ekstrand <[email protected]>
On Thu, Apr 6, 2017 at 8:56 PM, Ben Widawsky <[email protected]> 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 <[email protected]> > Signed-off-by: Ben Widawsky <[email protected]> > --- > 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 > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev >
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
