From patchwork Sat Jun  9 09:49:17 2012
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: wpa_supplicant: clear bssid state on deauth
Date: Fri, 08 Jun 2012 23:49:17 -0000
From: Jouni Malinen <j@w1.fi>
X-Patchwork-Id: 163906
Message-Id: <20120609094917.GE4128@w1.fi>
To: hostap@lists.shmoo.com

On Wed, May 30, 2012 at 09:32:48AM +0300, Arik Nemtsov wrote:
> When de-authenticated by an AP, the state of wpa_s should be cleared,
> especially the previous bssid. Otherwise, we will use re-assoc packets
> when associating to the same AP. This is not according to spec, and some
> APs refuse to association.

Would you be able to identify such an AP that rejects the reassociation
in this type of case?

> I'm not 100% sure with this patch, I just tried to copy the relevant
> parts from sme_deauth(). But it definitely fixed some real world issues
> we encountered with a mac80211 based driver. With prev_bssid_set == 1,
> re-assoc packets are used in mac80211 during association.

sme_deauth() is used when the local end is forcing deauthentication, so
this may not be the ideal place to copy from.. I would assume that
setting wpa_s->sme.prev_bssid_set = 0 is the only part that is really
needed here.

> diff --git a/wpa_supplicant/events.c b/wpa_supplicant/events.c
> @@ -2246,6 +2246,18 @@ void wpa_supplicant_event(void *ctx, enum wpa_event_type event,
> +
> +#ifdef CONFIG_AP
> +			if (wpa_s->ap_iface == NULL)
> +#endif /* CONFIG_AP */
> +			{
> +#ifdef CONFIG_SME
> +				wpa_s->sme.prev_bssid_set = 0;
> +#endif /* CONFIG_SME */
> +				wpa_supplicant_set_state(wpa_s,
> +							 WPA_DISCONNECTED);
> +				os_memset(wpa_s->bssid, 0, ETH_ALEN);
> +			}
>  		}
>  #ifdef CONFIG_AP
>  		if (wpa_s->ap_iface && data && data->deauth_info.addr) {

Those added lines are used only for deauthentication case. Why would
this not be done for disassociation? What is the purpose of
set_sta(DISCONNECTED) and clearing of wpa_s->bssid here?

Something like this could be better way of achieving the main goal of
not using reassociation:




Could you please confirm that this resolves the issue you've seen?


diff --git a/wpa_supplicant/events.c b/wpa_supplicant/events.c
index d09be6c..e7dfa4e 100644
--- a/wpa_supplicant/events.c
+++ b/wpa_supplicant/events.c
@@ -121,6 +121,9 @@ void wpa_supplicant_mark_disassoc(struct wpa_supplicant *wpa_s)
 	bssid_changed = !is_zero_ether_addr(wpa_s->bssid);
 	os_memset(wpa_s->bssid, 0, ETH_ALEN);
 	os_memset(wpa_s->pending_bssid, 0, ETH_ALEN);
+#ifdef CONFIG_SME
+	wpa_s->sme.prev_bssid_set = 0;
+#endif /* CONFIG_SME */
 #ifdef CONFIG_P2P
 	os_memset(wpa_s->go_dev_addr, 0, ETH_ALEN);
 #endif /* CONFIG_P2P */