mesa: use signed temporary variable to store _ColorDrawBufferIndexes
authorEmil Velikov <emil.l.velikov@gmail.com>
Sun, 12 Jan 2014 22:42:05 +0000 (22:42 +0000)
committerEmil Velikov <emil.l.velikov@gmail.com>
Wed, 15 Jan 2014 14:31:04 +0000 (14:31 +0000)
commit10368e1446e3b537c1dc4cb536994a4d01cfd2f0
tree1659c4357cfb22bd7d18026e936e0314daf2ab28
parent716b512dcf1d28eb9cafb31773aec68c4fd58122
mesa: use signed temporary variable to store _ColorDrawBufferIndexes

_ColorDrawBufferIndexes is defined as GLint* and using a GLuint*
will result in the first part of the conditional to be evaluated to
true always.

Unintentionally introduced by the following commit, this will result
in a driver segfault if one is using an old version of the piglit test

    bin/clearbuffer-mixed-format -auto -fbo

commit 03d848ea1003abefd8fe51a5b4a780527cd852af
Author: Marek Olšák <marek.olsak@amd.com>
Date:   Wed Dec 4 00:27:20 2013 +0100

    mesa: fix interpretation of glClearBuffer(drawbuffer)

    This corresponding piglit tests supported this incorrect behavior instead of
    pointing at it.

Cc: Marek Olšák <marek.olsak@amd.com>
Cc: 10.0 9.2 9.1 <mesa-stable@lists.freedesktop.org>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
src/mesa/main/clear.c