Don't use _mesa_swizzle_ubyte_image if all three texture formats

involved in _mesa_textore_argb/rgba8888 (source, base-internal,
destination) differ. _mesa_swizzle_ubyte_image knows only about two of
the formats and can't handle the case that all three are different
correctly. Fixes https://bugs.freedesktop.org/show_bug.cgi?id=3111.

1 files changed, 12 insertions, 0 deletions

diff --git a/src/mesa/main/texstore.c b/src/mesa/main/texstore.c
index e4c3f6651d..d46b23c803 100644
--- a/src/mesa/main/texstore.c
+++ b/src/mesa/main/texstore.c
@@ -1079,6 +1079,12 @@ _mesa_texstore_rgba8888(STORE_PARAMS)
 	    srcType == GL_UNSIGNED_BYTE && 
 	    dstFormat == &_mesa_texformat_rgba8888 &&
 	    littleEndian &&
+	    /* Three texture formats involved: srcFormat,
+	     * baseInternalFormat and destFormat (GL_RGBA). Only two
+	     * may differ. _mesa_swizzle_ubyte_image can't handle two
+	     * propagations at once correctly. */
+	    (srcFormat == baseInternalFormat ||
+	     baseInternalFormat == GL_RGBA) &&
 	    can_swizzle(srcFormat)) {
       GLubyte dstmap[4];
 
@@ -1252,6 +1258,12 @@ _mesa_texstore_argb8888(STORE_PARAMS)
 	    dstFormat == &_mesa_texformat_argb8888 &&
 	    srcType == GL_UNSIGNED_BYTE && 
 	    littleEndian &&
+	    /* Three texture formats involved: srcFormat,
+	     * baseInternalFormat and destFormat (GL_RGBA). Only two
+	     * may differ. _mesa_swizzle_ubyte_image can't handle two
+	     * propagations at once correctly. */
+	    (srcFormat == baseInternalFormat ||
+	     baseInternalFormat == GL_RGBA) &&
 	    can_swizzle(srcFormat)) {
 
       GLubyte dstmap[4];