diff options
author | Arik Nemtsov <arik@wizery.com> | 2011-08-14 06:17:37 -0400 |
---|---|---|
committer | Luciano Coelho <coelho@ti.com> | 2011-08-22 05:35:31 -0400 |
commit | cf42039f33c8c7c12f19390661eb00ba47b96f91 (patch) | |
tree | 2607757b8d884bf99860788dc6cf61aebc205e65 /drivers/net/wireless/wl12xx/main.c | |
parent | 9b17f1b371c5aa5179b3e5392bc22132a3371da4 (diff) |
wl12xx: set the AP-started flag only after setting keys
This fix eliminates a potential race between starting the AP role
and setting encryption keys.
Signed-off-by: Arik Nemtsov <arik@wizery.com>
Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
Diffstat (limited to 'drivers/net/wireless/wl12xx/main.c')
-rw-r--r-- | drivers/net/wireless/wl12xx/main.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/wireless/wl12xx/main.c b/drivers/net/wireless/wl12xx/main.c index ad0b5a163b9d..a23b394291f2 100644 --- a/drivers/net/wireless/wl12xx/main.c +++ b/drivers/net/wireless/wl12xx/main.c | |||
@@ -3140,12 +3140,12 @@ static void wl1271_bss_info_changed_ap(struct wl1271 *wl, | |||
3140 | if (ret < 0) | 3140 | if (ret < 0) |
3141 | goto out; | 3141 | goto out; |
3142 | 3142 | ||
3143 | set_bit(WL1271_FLAG_AP_STARTED, &wl->flags); | ||
3144 | wl1271_debug(DEBUG_AP, "started AP"); | ||
3145 | |||
3146 | ret = wl1271_ap_init_hwenc(wl); | 3143 | ret = wl1271_ap_init_hwenc(wl); |
3147 | if (ret < 0) | 3144 | if (ret < 0) |
3148 | goto out; | 3145 | goto out; |
3146 | |||
3147 | set_bit(WL1271_FLAG_AP_STARTED, &wl->flags); | ||
3148 | wl1271_debug(DEBUG_AP, "started AP"); | ||
3149 | } | 3149 | } |
3150 | } else { | 3150 | } else { |
3151 | if (test_bit(WL1271_FLAG_AP_STARTED, &wl->flags)) { | 3151 | if (test_bit(WL1271_FLAG_AP_STARTED, &wl->flags)) { |