diff options
author | Nanley Chery <nanley.g.chery@intel.com> | 2016-02-05 16:21:33 -0800 |
---|---|---|
committer | Nanley Chery <nanley.g.chery@intel.com> | 2016-02-09 15:13:07 -0800 |
commit | 605832736a6d9427ad894d403cceeb74a5b18dc1 (patch) | |
tree | 82eef6c48ec62dc165d3dc843007fc3dcc6e58e8 /src/mesa/main/api_arrayelt.h | |
parent | 55d56d34e0535baa2c7e1e1d8f1be11593a07fa8 (diff) |
mesa/readpix: Clip ReadPixels() area to the ReadBuffer's
The fast path for Intel's ReadPixels() unintentionally omits clipping
the specified area to a valid one. Rather than clip in various
corner-cases, perform this operation in the API validation stage.
The bug in intel_readpixels_tiled_memcpy() showed itself when the winsys
ReadBuffer's height was smaller than the one specified by ReadPixels().
yoffset became negative, which was an invalid input for tiled_to_linear().
v2: Move clipping to validation stage (Jason)
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92193
Reported-by: Marta Löfstedt <marta.lofstedt@intel.com>
Cc: "11.0 11.1" <mesa-stable@lists.freedesktop.org>
Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
Diffstat (limited to 'src/mesa/main/api_arrayelt.h')
0 files changed, 0 insertions, 0 deletions