From: Michel Dänzer Date: Fri, 24 Apr 2020 09:40:14 +0000 (+0200) Subject: mesa: Skip 3-byte array formats in _mesa_array_format_flip_channels X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c50bbfa0ab513a771167b3885fdbb2b5c75d2384;p=mesa.git mesa: Skip 3-byte array formats in _mesa_array_format_flip_channels Byte swapping makes no sense for 3-byte formats: Swapping the order of 2 or 4 bytes at a time would inevitably result in bytes getting mixed up between neighbouring pixels. Fixes crash with a debugging build on a big endian machine due hitting the unreachable() at the end of the function. Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/2665 Reviewed-by: Marek Olšák Part-of: --- diff --git a/src/mesa/main/formats.c b/src/mesa/main/formats.c index 114ea5218be..0bca52fde50 100644 --- a/src/mesa/main/formats.c +++ b/src/mesa/main/formats.c @@ -404,7 +404,7 @@ _mesa_array_format_flip_channels(mesa_array_format format) num_channels = _mesa_array_format_get_num_channels(format); _mesa_array_format_get_swizzle(format, swizzle); - if (num_channels == 1) + if (num_channels == 1 || num_channels == 3) return format; if (num_channels == 2) {