Commit 079908f1 authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman

i915_gem: return -EFAULT if copy_to_user fails

commit c877cdce upstream.

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: default avatarDan Carpenter <error27@gmail.com>
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 9cd74cb5
...@@ -3667,6 +3667,7 @@ i915_gem_execbuffer(struct drm_device *dev, void *data, ...@@ -3667,6 +3667,7 @@ i915_gem_execbuffer(struct drm_device *dev, void *data,
if (ret != 0) { if (ret != 0) {
DRM_ERROR("copy %d cliprects failed: %d\n", DRM_ERROR("copy %d cliprects failed: %d\n",
args->num_cliprects, ret); args->num_cliprects, ret);
ret = -EFAULT;
goto pre_mutex_err; goto pre_mutex_err;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment