Message ID | CAFk-A4nF8aoppuRQ68u82xFFuAiXMf_cBc+iPtJXH_yb7kZLrw@mail.gmail.com |
---|---|
State | Accepted |
Commit | 0a0c38f63d825b352deb819b32a0fb1203eb936c |
Headers | show |
On Fri, Jul 13, 2012 at 10:00:45PM +0900, Masashi Honma wrote: > There is a device driver causes Fig.2 case. > The device driver returns EINVAL for get_bssid() because it recognizes it has > already been disconnected. > When the wpa_supplicant received EINVAL, the wpa_supplicant recognized that > it's own BSSID is 00:00:00:00:00:00. > This patch solves this issue. Thanks, applied. Please include the Signed-hostap: tag in future contributions to avoid extra delays once I start being more strict about this getting included in every commit to hostap.git.
diff --git a/wpa_supplicant/events.c b/wpa_supplicant/events.c index 59b103e..5b7c14a 100644 --- a/wpa_supplicant/events.c +++ b/wpa_supplicant/events.c @@ -1523,9 +1523,15 @@ static void wpa_supplicant_event_assoc(struct wpa_supplicant *wpa_s, if (data && wpa_supplicant_event_associnfo(wpa_s, data) < 0) return; + if (wpa_drv_get_bssid(wpa_s, bssid) < 0) { + wpa_dbg(wpa_s, MSG_ERROR, "Failed to get BSSID"); + wpa_supplicant_disassociate( + wpa_s, WLAN_REASON_DEAUTH_LEAVING); + return; + } + wpa_supplicant_set_state(wpa_s, WPA_ASSOCIATED); - if (wpa_drv_get_bssid(wpa_s, bssid) >= 0 && - os_memcmp(bssid, wpa_s->bssid, ETH_ALEN) != 0) { + if (os_memcmp(bssid, wpa_s->bssid, ETH_ALEN) != 0) { wpa_dbg(wpa_s, MSG_DEBUG, "Associated to a new BSS: BSSID=" MACSTR, MAC2STR(bssid));