aboutsummaryrefslogtreecommitdiff
path: root/drivers/staging/ks7010/ks_wlan_net.c
diff options
context:
space:
mode:
authorTobin C. Harding <me@tobin.cc>2017-03-14 09:54:08 +1100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-03-14 08:08:15 +0800
commitf0b7d3a67bce75bc1e5910e30c3890ae01e3be04 (patch)
tree08f851960cb3b3b8bfed27b09ec65016ab189492 /drivers/staging/ks7010/ks_wlan_net.c
parentad1e187f41a6dc008e5057805dc713551108799a (diff)
staging: ks7010: fix checkpatch memset warning
Checkpatch emits WARNING: single byte memset is suspicious. Swapped 2nd/3rd argument? Call site in question is correct but is an unusual use of memset() to zero a single byte. The same can be achieved by assigning 0 directly to the memory location. Dereference pointer and assign 0 to that memory location. Use '\0' to make explicit that it is a char pointer. Signed-off-by: Tobin C. Harding <me@tobin.cc> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/ks7010/ks_wlan_net.c')
-rw-r--r--drivers/staging/ks7010/ks_wlan_net.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/staging/ks7010/ks_wlan_net.c
index 9fa1ee7255ca..d07b724a3442 100644
--- a/drivers/staging/ks7010/ks_wlan_net.c
+++ b/drivers/staging/ks7010/ks_wlan_net.c
@@ -2431,7 +2431,7 @@ static int ks_wlan_data_read(struct net_device *dev,
return 0;
}
read_length = 0;
- memset(extra, 0, 1);
+ *extra = '\0';
dwrq->length = 0;
return 0;
}