drm/vc4: Fix an integer overflow in temporary allocation layout.
authorEric Anholt <eric@anholt.net>
Tue, 17 Jan 2017 20:20:49 +0000 (07:20 +1100)
committerBen Hutchings <ben@decadent.org.uk>
Sat, 28 Jan 2017 16:11:16 +0000 (16:11 +0000)
commit07c68d3f84fe2d4cff5e48383dc291406e0fc633
tree383da3059573fe4f688e444d1a34ce26b3cb57d7
parent5ccbf3afdd35aa3ae16d718084698d554223a82f
drm/vc4: Fix an integer overflow in temporary allocation layout.

We copy the unvalidated ioctl arguments from the user into kernel
temporary memory to run the validation from, to avoid a race where the
user updates the unvalidate contents in between validating them and
copying them into the validated BO.

However, in setting up the layout of the kernel side, we failed to
check one of the additions (the roundup() for shader_rec_offset)
against integer overflow, allowing a nearly MAX_UINT value of
bin_cl_size to cause us to under-allocate the temporary space that we
then copy_from_user into.

Reported-by: Murray McAllister <murray.mcallister@insomniasec.com>
Signed-off-by: Eric Anholt <eric@anholt.net>
Fixes: d5b1a78a772f ("drm/vc4: Add support for drawing 3D frames.")
Gbp-Pq: Topic bugfix/arm
Gbp-Pq: Name drm-vc4-fix-an-integer-overflow-in-temporary-allocation-layout.patch
drivers/gpu/drm/vc4/vc4_gem.c