diff options
author | Tobin C. Harding <me@tobin.cc> | 2017-03-14 09:54:08 +1100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-14 08:08:15 +0800 |
commit | f0b7d3a67bce75bc1e5910e30c3890ae01e3be04 (patch) | |
tree | 08f851960cb3b3b8bfed27b09ec65016ab189492 /drivers/staging/ks7010/ks_wlan_net.c | |
parent | ad1e187f41a6dc008e5057805dc713551108799a (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.c | 2 |
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; } |