diff options
| author | Nanley Chery <[email protected]> | 2016-02-05 16:21:33 -0800 |
|---|---|---|
| committer | Nanley Chery <[email protected]> | 2016-02-09 15:13:07 -0800 |
| commit | 605832736a6d9427ad894d403cceeb74a5b18dc1 (patch) | |
| tree | 82eef6c48ec62dc165d3dc843007fc3dcc6e58e8 /src/glx/eval.c | |
| 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 <[email protected]>
Cc: "11.0 11.1" <[email protected]>
Signed-off-by: Nanley Chery <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Reviewed-by: Brian Paul <[email protected]>
Diffstat (limited to 'src/glx/eval.c')
0 files changed, 0 insertions, 0 deletions