diff options
| author | Dan Carpenter | 2010-06-04 12:24:13 +0200 |
|---|---|---|
| committer | Dave Airlie | 2010-06-08 01:33:41 +0200 |
| commit | 9b8eb4d14767209c83087063352cd04266ecdfd1 (patch) | |
| tree | 7b98bf40cd556d29c0a673a1c3d141d08635b455 /drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | |
| parent | drm/drm_crtc: return -EFAULT on copy_to_user errors (diff) | |
| download | kernel-qcow2-linux-9b8eb4d14767209c83087063352cd04266ecdfd1.tar.gz kernel-qcow2-linux-9b8eb4d14767209c83087063352cd04266ecdfd1.tar.xz kernel-qcow2-linux-9b8eb4d14767209c83087063352cd04266ecdfd1.zip | |
drm/vmwgfx: return -EFAULT for copy_to_user errors
copy_to/from_user() returns the number of bytes remaining to be copied
but we want to return a negative error code here. This gets returned to
userspace.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c')
| -rw-r--r-- | drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c index bdd67cf83315..8e396850513c 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c @@ -644,6 +644,7 @@ int vmw_execbuf_ioctl(struct drm_device *dev, void *data, ret = copy_from_user(cmd, user_cmd, arg->command_size); if (unlikely(ret != 0)) { + ret = -EFAULT; DRM_ERROR("Failed copying commands.\n"); goto out_commit; } |