diff options
author | Dan Carpenter <error27@gmail.com> | 2010-06-23 13:03:01 -0400 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2010-09-06 18:09:54 -0400 |
commit | c877cdce93a44eea96f6cf7fc04be7d0372db2be (patch) | |
tree | 6d14849160a0bfe37f05f3fed41357246a1803a8 /drivers/gpu | |
parent | 9927a403ca8c97798129953fa9cbb5dc259c7cb9 (diff) |
i915: return -EFAULT if copy_to_user fails
copy_to_user() returns the number of bytes remaining to be copied and
I'm pretty sure we want to return a negative error code here.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: stable@kernel.org
Diffstat (limited to 'drivers/gpu')
-rw-r--r-- | drivers/gpu/drm/i915/i915_gem.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 68526f467c87..748c26340c35 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b/drivers/gpu/drm/i915/i915_gem.c | |||
@@ -3588,6 +3588,7 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data, | |||
3588 | if (ret != 0) { | 3588 | if (ret != 0) { |
3589 | DRM_ERROR("copy %d cliprects failed: %d\n", | 3589 | DRM_ERROR("copy %d cliprects failed: %d\n", |
3590 | args->num_cliprects, ret); | 3590 | args->num_cliprects, ret); |
3591 | ret = -EFAULT; | ||
3591 | goto pre_mutex_err; | 3592 | goto pre_mutex_err; |
3592 | } | 3593 | } |
3593 | } | 3594 | } |