mesa/pbo: Handle zero width, height or depth when validating access
authorNeil Roberts <neil@linux.intel.com>
Wed, 2 Sep 2015 10:29:16 +0000 (11:29 +0100)
committerNeil Roberts <neil@linux.intel.com>
Thu, 3 Sep 2015 16:00:54 +0000 (17:00 +0100)
commitce181aea6cb5353181add7b5aca3c0b196a9b513
treead254471ca9c5148d462259c4379652753849f82
parent30e84530a097278c7cf01c0491dba5866510c4c5
mesa/pbo: Handle zero width, height or depth when validating access

It's legal to call glTexSubImage with zero values for the width,
height or depth. Previously this was breaking the PBO access
validation because it tries to work out the last pixel accessed by
getting the pixel at height-1 and depth-1 which would end up with
bogus values.

This was causing GL errors to be generated during the Piglit
texsubimage test, although the test was passing anyway.

v2: Also check for width == 0. Don't validate the start pointer if any
    of the dimensions are zero.
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
src/mesa/main/pbo.c