[Mesa-dev] [PATCH] mesa: reset the source packing when creating temp transfer image

[Anuj Phogat]

On Wed, Jul 1, 2015 at 12:18 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> Commit 4b249d2ee (mesa: Handle transferOps in texstore_rgba) introduced
> proper transferops handling, but in updating the source to the newly
> allocated temporary image neglected to reset the source packing. Set it
> to the default which should be appropriate for the floats used.
>
> Fixes: 4b249d2ee (mesa: Handle transferOps in texstore_rgba)
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91173
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> Cc: "10.5 10.6" <mesa-stable at lists.freedesktop.org>
> ---
>  src/mesa/main/texstore.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/main/texstore.c b/src/mesa/main/texstore.c
> index 1525205..37c0569 100644
> --- a/src/mesa/main/texstore.c
> +++ b/src/mesa/main/texstore.c
> @@ -787,6 +787,7 @@ texstore_rgba(TEXSTORE_PARAMS)
>        srcType = GL_FLOAT;
>        srcRowStride = srcWidth * 4 * sizeof(float);
>        srcMesaFormat = RGBA32_FLOAT;
> +      srcPacking = &ctx->DefaultPacking;
>     }
>
>     src = (GLubyte *)
> --
> 2.3.6
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>