lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

staging: vt6656: implement missing brackets

Browse Source 
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>
This commit is contained in:
Vasiliy Kulikov
2010-10-29 00:02:11 +04:00
committed by Greg Kroah-Hartman
parent 705059a670
commit eacd121c3d
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
drivers/staging/vt6656/main_usb.c
View File

@@ -1675,13 +1675,14 @@ static int device_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) {
{ {
char essid[IW_ESSID_MAX_SIZE+1]; char essid[IW_ESSID_MAX_SIZE+1];
if (wrq->u.essid.pointer) if (wrq->u.essid.pointer) {
rc = iwctl_giwessid(dev, NULL, rc = iwctl_giwessid(dev, NULL,
&(wrq->u.essid), essid); &(wrq->u.essid), essid);
if (copy_to_user(wrq->u.essid.pointer, if (copy_to_user(wrq->u.essid.pointer,
essid, essid,
wrq->u.essid.length) ) wrq->u.essid.length) )
rc = -EFAULT; rc = -EFAULT;
}
} }
break; break;