diff options
author | dan.carpenter@oracle.com <dan.carpenter@oracle.com> | 2017-07-19 12:51:40 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-07-29 02:55:40 +0300 |
commit | 39ae5f1e4b86333b2ca97a278a93640d61fcf77d (patch) | |
tree | 20f26761cbd7d6d441b6b29fa02fbb74a8c245ac /drivers/staging/pi433 | |
parent | a7b1ba23556215952099274a0f008643de832f70 (diff) | |
download | linux-39ae5f1e4b86333b2ca97a278a93640d61fcf77d.tar.xz |
staging: pi433: return -EFAULT if copy_to_user() fails
copy_to_user() to user returns the number of bytes that weren't copied,
but we should be returning -EFAULT to the user.
Fixes: 874bcba65f9a ("staging: pi433: New driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/pi433')
-rw-r--r-- | drivers/staging/pi433/pi433_if.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/pi433/pi433_if.c b/drivers/staging/pi433/pi433_if.c index d9328ce5ec1d..416d7ad8e9d7 100644 --- a/drivers/staging/pi433/pi433_if.c +++ b/drivers/staging/pi433/pi433_if.c @@ -689,7 +689,7 @@ pi433_read(struct file *filp, char __user *buf, size_t size, loff_t *f_pos) { retval = copy_to_user(buf, device->rx_buffer, bytes_received); if (retval) - return retval; + return -EFAULT; } return bytes_received; |