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 /drivers/remoteproc | |
| 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>
Diffstat (limited to 'drivers/remoteproc')
| -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') |