diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-03-20 06:21:45 +0300 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2006-03-24 00:16:57 +0300 |
| commit | 54b85f489bdfafc9306dfcc21e0d2687c34c3b34 (patch) | |
| tree | c141cbcf5fbad9f7e68fdf12567871c049736b34 /drivers/net/wireless/hostap/hostap_ap.c | |
| parent | 971d1e69229f3dfc6dd233b8f4b82df2c35eedd8 (diff) | |
| download | linux-54b85f489bdfafc9306dfcc21e0d2687c34c3b34.tar.xz | |
[PATCH] hostap: Fix ap_add_sta() return value verification
The Coverity checker (CID: 273) spotted this inconsequent NULL checking
(unconditionally dereferencing directly after checking for NULL
isn't a good idea). Return immediately to avoid this.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Jouni Malinen <jkmaline@cc.hut.fi>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/hostap/hostap_ap.c')
| -rw-r--r-- | drivers/net/wireless/hostap/hostap_ap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/hostap/hostap_ap.c b/drivers/net/wireless/hostap/hostap_ap.c index 753a1de6664b..06c3fa32b310 100644 --- a/drivers/net/wireless/hostap/hostap_ap.c +++ b/drivers/net/wireless/hostap/hostap_ap.c @@ -3141,7 +3141,7 @@ int hostap_add_sta(struct ap_data *ap, u8 *sta_addr) if (ret == 1) { sta = ap_add_sta(ap, sta_addr); if (!sta) - ret = -1; + return -1; sta->flags = WLAN_STA_AUTH | WLAN_STA_ASSOC; sta->ap = 1; memset(sta->supported_rates, 0, sizeof(sta->supported_rates)); |