[PATCH v2] drm: remove redundant code form drm_ioc32.c

2015-07-03 Thread Jarkko Sakkinen
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

[PATCH v2] drm: remove redundant code form drm_ioc32.c

2015-07-03 Thread Daniel Vetter
On Fri, Jul 03, 2015 at 02:17:29PM +0300, Jarkko Sakkinen wrote: > 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 acc

[PATCH v2] drm: remove redundant code form drm_ioc32.c

2015-07-03 Thread Daniel Vetter
On Fri, Jul 03, 2015 at 02:17:29PM +0300, Jarkko Sakkinen wrote: > 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 acc