[Mesa-dev] [PATCH mesa] i965: Check result of make_surface() for miptree_create

[Ian Romanick]

On 05/17/2018 05:05 AM, Eric Engestrom wrote:
> From: Andrea Azzarone <azzaronea at 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.engestrom at 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);
>