Thread (1354 messages) 1354 messages, 168 authors, 2017-04-12

Re: [PATCH 3/3] staging: wilc1000: Reduce scope for a few variables in mac_ioctl()

From: walter harms <hidden>
Date: 2016-07-25 19:32:16
Also in: linux-wireless, lkml


Am 25.07.2016 21:17, schrieb Lino Sanfilippo:
quoted
 
-			if (strncasecmp(buff, "RSSI", length) = 0) {
+			if (strncasecmp(buff, "RSSI", 0) = 0) {
+				s8 rssi;
+
Um, please think a second about if it makes any sense at all to compare 
zero chars of two strings.

Lino
also:
the switch has only one case and default.

re,
 wh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help