drm: remove redundant code form drm_ioc32.c
authorJarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Fri, 3 Jul 2015 11:17:29 +0000 (14:17 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 3 Jul 2015 11:58:30 +0000 (13:58 +0200)
commit1f96beec7a44f46bf277e2050a7a5a0308831d61
tree45bf3931c1ccab48857b73e1ecc3830e8ad99ae2
parente4f31ad2b7138db449310c9b63d402a29dc0e1c9
drm: remove redundant code form drm_ioc32.c

The compat ioctl handler ends up calling access_ok() twice: first
indirectly inside compat_alloc_user_space() and then after returning
from that function. This patch fixes issue.

v2: there were three invalid removals of access_ok() that I've fixed.
Also went through all the changes couple of times and verified that
access_ok() is only removed when the buffer is allocated with
compat_alloc_user_space(). My deepest apologies for this kind of
sloppiness!

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/drm_ioc32.c