diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-06-04 12:24:13 +0200 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2010-06-08 09:33:41 +1000 |
| commit | 9b8eb4d14767209c83087063352cd04266ecdfd1 (patch) | |
| tree | 7b98bf40cd556d29c0a673a1c3d141d08635b455 /drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | |
| parent | e902a358c753b93245083201c02312a580cf13d4 (diff) | |
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; } |