diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-09-25 10:05:33 +0300 |
---|---|---|
committer | Ohad Ben-Cohen <ohad@wizery.com> | 2012-09-30 11:44:53 +0200 |
commit | bec109a430e8c67bae1743f7e71898283234a77f (patch) | |
tree | 181eb8689c59f3fa6a63f28468591fceb93a68c9 | |
parent | ae768d5fac18cd4b3ff51a4860b89fabba8ccc65 (diff) | |
download | op-kernel-dev-bec109a430e8c67bae1743f7e71898283234a77f.zip op-kernel-dev-bec109a430e8c67bae1743f7e71898283234a77f.tar.gz |
remoteproc: return -EFAULT on copy_from_user failure
copy_from_user() returns the number of bytes remaining to be copied, but
we want to return an error code here.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Ohad Ben-Cohen <ohad@wizery.com>
-rw-r--r-- | drivers/remoteproc/remoteproc_debugfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/remoteproc/remoteproc_debugfs.c b/drivers/remoteproc/remoteproc_debugfs.c index ea90a56..157a573 100644 --- a/drivers/remoteproc/remoteproc_debugfs.c +++ b/drivers/remoteproc/remoteproc_debugfs.c @@ -161,7 +161,7 @@ rproc_recovery_write(struct file *filp, const char __user *user_buf, ret = copy_from_user(buf, user_buf, count); if (ret) - return ret; + return -EFAULT; /* remove end of line */ if (buf[count - 1] == '\n') |