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.manol...@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 |
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
