diff options
author | Dan Carpenter <error27@gmail.com> | 2010-06-04 06:39:51 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-06-04 16:38:58 -0400 |
commit | 2d98bb22c33b033182edd3a3c6b4c4e1d091db4d (patch) | |
tree | 9e8995b79993fc279383ee713ee6f9fbb7d5642d /drivers/staging | |
parent | c60e55f30a4dac15db51b398c3bd94e4cfbf743a (diff) |
Staging: rc2860: return -EFAULT on copy_to_user errors
copy_to_user() returns the number of bytes remaining but we want to
return a negative error code. This is in the ioctl handler and the
error code gets passed to userspace.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/rt2860/sta_ioctl.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/staging/rt2860/sta_ioctl.c b/drivers/staging/rt2860/sta_ioctl.c index 112da7a6c417..6b8268d3dc75 100644 --- a/drivers/staging/rt2860/sta_ioctl.c +++ b/drivers/staging/rt2860/sta_ioctl.c | |||
@@ -2522,6 +2522,8 @@ int rt28xx_sta_ioctl(IN struct net_device *net_dev, | |||
2522 | Status = | 2522 | Status = |
2523 | copy_to_user(erq->pointer, pAd->nickname, | 2523 | copy_to_user(erq->pointer, pAd->nickname, |
2524 | erq->length); | 2524 | erq->length); |
2525 | if (Status) | ||
2526 | Status = -EFAULT; | ||
2525 | break; | 2527 | break; |
2526 | } | 2528 | } |
2527 | case SIOCGIWRATE: /*get default bit rate (bps) */ | 2529 | case SIOCGIWRATE: /*get default bit rate (bps) */ |