diff options
author | Vasiliy Kulikov <segooon@gmail.com> | 2010-10-28 16:02:11 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-11-09 17:05:23 -0500 |
commit | eacd121c3d0b74220aa6a91223e0adf7d5ec2497 (patch) | |
tree | 2b260ebfc7d54b746132daf0e217567933e7a5a9 | |
parent | 705059a670f3af2b37695e82de0ee58e75e656ed (diff) |
staging: vt6656: implement missing brackets
Identation says that copy_to_user() should be called only iff
wrq->u.essid.pointer is not zero. Also it is useless to call copy_to_user(0, ...).
Signed-off-by: Vasiliy Kulikov <segooon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/staging/vt6656/main_usb.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/vt6656/main_usb.c b/drivers/staging/vt6656/main_usb.c index e992d5d9e15b..7cc3d2407d1b 100644 --- a/drivers/staging/vt6656/main_usb.c +++ b/drivers/staging/vt6656/main_usb.c | |||
@@ -1675,13 +1675,14 @@ static int device_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) { | |||
1675 | 1675 | ||
1676 | { | 1676 | { |
1677 | char essid[IW_ESSID_MAX_SIZE+1]; | 1677 | char essid[IW_ESSID_MAX_SIZE+1]; |
1678 | if (wrq->u.essid.pointer) | 1678 | if (wrq->u.essid.pointer) { |
1679 | rc = iwctl_giwessid(dev, NULL, | 1679 | rc = iwctl_giwessid(dev, NULL, |
1680 | &(wrq->u.essid), essid); | 1680 | &(wrq->u.essid), essid); |
1681 | if (copy_to_user(wrq->u.essid.pointer, | 1681 | if (copy_to_user(wrq->u.essid.pointer, |
1682 | essid, | 1682 | essid, |
1683 | wrq->u.essid.length) ) | 1683 | wrq->u.essid.length) ) |
1684 | rc = -EFAULT; | 1684 | rc = -EFAULT; |
1685 | } | ||
1685 | } | 1686 | } |
1686 | break; | 1687 | break; |
1687 | 1688 | ||