On 05/17/2018 05:05 AM, Eric Engestrom wrote: > From: Andrea Azzarone <azzaro...@gmail.com> > > Since make_surface() can fail we need to check the result before > dereferencing it. > > Bug: https://github.com/mesa3d/mesa/pull/5 > Bug: https://bugs.launchpad.net/ubuntu/+source/gnome-shell/+bug/1760415 > Reviewed-by: Eric Engestrom <eric.engest...@intel.com> > --- > Andrea: We don't use github, I only happened to notice your pull request :) > Next time you want to send us something, send it here :P > --- > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > index 67086ee6c0e8d6b6feb0..43687ea77abfe9989882 100644 > --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > @@ -718,6 +718,9 @@ miptree_create(struct brw_context *brw, > ISL_SURF_USAGE_DEPTH_BIT | ISL_SURF_USAGE_TEXTURE_BIT, > BO_ALLOC_BUSY, 0, NULL); > > + if (!mt) > + return NULL; > +
So... I sent this same patch a few months ago, but we decided to not land it. https://patchwork.freedesktop.org/patch/195626/ > if (needs_separate_stencil(brw, mt, format) && > !make_separate_stencil_surface(brw, mt)) { > intel_miptree_release(&mt); > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
