diff options
author | Eric Anholt <[email protected]> | 2012-11-13 13:39:37 -0800 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2012-11-17 12:55:54 -0800 |
commit | 5c99697f74c68f7e170564d791b95d986a18774a (patch) | |
tree | df53cbb8f4e783d09d10e14ff73371390e9b815a /src/mesa/state_tracker | |
parent | df3361df01e35aae9f379f0687490ba409841cfb (diff) |
mesa: Fix segfault on reading from a missing color read buffer.
The diff looks funny, but it's moving the integer vs non-integer check
below the _mesa_source_buffer_exists() check that ensures
_ColorReadBuffer is non-null, so we get a GL_INVALID_OPERATION instead
of a segfault. This looks like it had regressed in the
_mesa_error_check_format_and_type() changes, which removed the first of
the two duplicated checks for the source buffer. Fixes segfault in the
new piglit ARB_framebuffer_object/negative-readpixels-no-rb.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=45877
NOTE: This is a candidate for the stable branches.
Reviewed-by: Brian Paul <[email protected]>
Diffstat (limited to 'src/mesa/state_tracker')
0 files changed, 0 insertions, 0 deletions