diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-09-25 03:05:33 -0400 |
---|---|---|
committer | Ohad Ben-Cohen <ohad@wizery.com> | 2012-09-30 05:44:53 -0400 |
commit | bec109a430e8c67bae1743f7e71898283234a77f (patch) | |
tree | 181eb8689c59f3fa6a63f28468591fceb93a68c9 | |
parent | ae768d5fac18cd4b3ff51a4860b89fabba8ccc65 (diff) |
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 ea90a5643db3..157a57309601 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, | |||
161 | 161 | ||
162 | ret = copy_from_user(buf, user_buf, count); | 162 | ret = copy_from_user(buf, user_buf, count); |
163 | if (ret) | 163 | if (ret) |
164 | return ret; | 164 | return -EFAULT; |
165 | 165 | ||
166 | /* remove end of line */ | 166 | /* remove end of line */ |
167 | if (buf[count - 1] == '\n') | 167 | if (buf[count - 1] == '\n') |