staging: ks7010: fix checkpatch memset warning
authorTobin C. Harding <me@tobin.cc>
Mon, 13 Mar 2017 22:54:08 +0000 (09:54 +1100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Mar 2017 00:08:15 +0000 (08:08 +0800)
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>
drivers/staging/ks7010/ks_wlan_net.c

index 9fa1ee7255cae051c93434ecbf9e262f42a87cae..d07b724a3442dfe565518eb8268e0c878b02ef30 100644 (file)
@@ -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;
        }