diff options
author | Adam Jackson <[email protected]> | 2016-05-24 15:45:11 -0400 |
---|---|---|
committer | Adam Jackson <[email protected]> | 2016-06-08 14:39:46 -0400 |
commit | a1c5cd426c0381124f7c320d5a7b760a9a36af75 (patch) | |
tree | a043f8fea0957398880915ea732336b10232746d /src/mapi/mapi_glapi.c | |
parent | 26b69ad250ee23e70831626a88f70f6ddf2e1bcc (diff) |
glapi/glx: Add overflow checks to the client-side indirect code
Coverity complains that the computed sizes can lead to negative lengths
passed to memcpy. If that happens we've been handed invalid arguments
anyway, so just bomb out.
The funky "0%s" is because the size string for the variable-length part
of the request is of the form "+ safe_pad() ...", and a unary + would
coerce the result to always be positive, defeating the overflow check.
Signed-off-by: Adam Jackson <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/mapi/mapi_glapi.c')
0 files changed, 0 insertions, 0 deletions