[Mesa-stable] [Mesa-dev] [PATCH 1/3] mesa/main: Move NULL pointer check.

Andres Gomez agomez at igalia.com
Thu Jul 6 14:35:52 UTC 2017


It looks like we could want this into -stable (?)

On Wed, 2017-06-14 at 19:33 +0300, Plamena Manolova wrote:
> In blit_framebuffer we're already doing a NULL
> pointer check for readFb and drawFb so it makes
> sense to do it before we actually use the pointers.
> 
> CID: 1412569
> Signed-off-by: Plamena Manolova <plamena.manolova at intel.com>
> ---
>  src/mesa/main/blit.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/src/mesa/main/blit.c b/src/mesa/main/blit.c
> index bb866b1..8bb3ba3 100644
> --- a/src/mesa/main/blit.c
> +++ b/src/mesa/main/blit.c
> @@ -349,12 +349,6 @@ blit_framebuffer(struct gl_context *ctx,
>  {
>     FLUSH_VERTICES(ctx, 0);
>  
> -   /* Update completeness status of readFb and drawFb. */
> -   _mesa_update_framebuffer(ctx, readFb, drawFb);
> -
> -   /* Make sure drawFb has an initialized bounding box. */
> -   _mesa_update_draw_buffer_bounds(ctx, drawFb);
> -
>     if (!readFb || !drawFb) {
>        /* This will normally never happen but someday we may want to
>         * support MakeCurrent() with no drawables.
> @@ -362,6 +356,12 @@ blit_framebuffer(struct gl_context *ctx,
>        return;
>     }
>  
> +   /* Update completeness status of readFb and drawFb. */
> +   _mesa_update_framebuffer(ctx, readFb, drawFb);
> +
> +   /* Make sure drawFb has an initialized bounding box. */
> +   _mesa_update_draw_buffer_bounds(ctx, drawFb);
> +
>     if (!no_error) {
>        const GLbitfield legalMaskBits = (GL_COLOR_BUFFER_BIT |
>                                          GL_DEPTH_BUFFER_BIT |


More information about the mesa-stable mailing list