summaryrefslogtreecommitdiff
path: root/drivers/net/wireless/marvell/mwifiex/main.h
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2016-01-28 22:58:28 +0100
committerKalle Valo <kvalo@codeaurora.org>2016-02-06 13:59:24 +0200
commit48dc5fb3ba53b20418de8514700f63d88c5de3a3 (patch)
tree1b9b1ea7d9fec869ac54c9eb353f2f48799e9e40 /drivers/net/wireless/marvell/mwifiex/main.h
parent97f1a17109272b9f060cbd6d1d5be41528643000 (diff)
hostap: avoid uninitialized variable use in hfa384x_get_rid
The driver reads a value from hfa384x_from_bap(), which may fail, and then assigns the value to a local variable. gcc detects that in in the failure case, the 'rlen' variable now contains uninitialized data: In file included from ../drivers/net/wireless/intersil/hostap/hostap_pci.c:220:0: drivers/net/wireless/intersil/hostap/hostap_hw.c: In function 'hfa384x_get_rid': drivers/net/wireless/intersil/hostap/hostap_hw.c:842:5: warning: 'rec' may be used uninitialized in this function [-Wmaybe-uninitialized] if (le16_to_cpu(rec.len) == 0) { This restructures the function as suggested by Russell King, to make it more readable and get more reliable error handling, by handling each failure mode using a goto. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to 'drivers/net/wireless/marvell/mwifiex/main.h')
0 files changed, 0 insertions, 0 deletions