On Thu, 2009-08-13 at 17:23 +0800, Zhu Yi wrote: > When we receive a successful status in CFG80211_SME_CONNECTED state, > it is a roam event. We should mark it as a success result. But there's a cfg80211_roamed() call for that? Can the driver not tell the difference? It also sends a different event (ROAMED rather than CONNECTED) to userspace. johannes > Cc: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > Signed-off-by: Zhu Yi <yi.zhu@xxxxxxxxx> > --- > net/wireless/sme.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/net/wireless/sme.c b/net/wireless/sme.c > index 8e2ef54..1aa1190 100644 > --- a/net/wireless/sme.c > +++ b/net/wireless/sme.c > @@ -393,7 +393,8 @@ void __cfg80211_connect_result(struct net_device *dev, const u8 *bssid, > } > > if (status == WLAN_STATUS_SUCCESS && > - wdev->sme_state == CFG80211_SME_IDLE) > + (wdev->sme_state == CFG80211_SME_IDLE || > + wdev->sme_state == CFG80211_SME_CONNECTED)) > goto success; > > if (wdev->sme_state != CFG80211_SME_CONNECTING)
Attachment:
signature.asc
Description: This is a digitally signed message part
