cfg80211: Allow reassociation to be requested with internal SME
authorJouni Malinen <jouni@qca.qualcomm.com>
Tue, 29 Mar 2016 10:53:28 +0000 (13:53 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 6 Apr 2016 13:09:28 +0000 (15:09 +0200)
If the user space issues a NL80211_CMD_CONNECT with
NL80211_ATTR_PREV_BSSID when there is already a connection, allow this
to proceed as a reassociation instead of rejecting the new connect
command with EALREADY.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
[validate prev_bssid]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/nl80211.c
net/wireless/sme.c

index 4f89e2dbb70e0fa906cb82f70829bc7abe98c401..4f45a2913104e42e692bde95a8e4ecb621767b15 100644 (file)
@@ -8151,7 +8151,8 @@ static int nl80211_connect(struct sk_buff *skb, struct genl_info *info)
        }
 
        wdev_lock(dev->ieee80211_ptr);
-       err = cfg80211_connect(rdev, dev, &connect, connkeys, NULL);
+       err = cfg80211_connect(rdev, dev, &connect, connkeys,
+                              connect.prev_bssid);
        wdev_unlock(dev->ieee80211_ptr);
        if (err)
                kzfree(connkeys);
index 65882d2777c0e9985122041d4674e5dd3dc1d067..1fba41676428ce5e3a54d797cc494d06e432ebaa 100644 (file)
@@ -492,8 +492,18 @@ static int cfg80211_sme_connect(struct wireless_dev *wdev,
        if (!rdev->ops->auth || !rdev->ops->assoc)
                return -EOPNOTSUPP;
 
-       if (wdev->current_bss)
-               return -EALREADY;
+       if (wdev->current_bss) {
+               if (!prev_bssid)
+                       return -EALREADY;
+               if (prev_bssid &&
+                   !ether_addr_equal(prev_bssid, wdev->current_bss->pub.bssid))
+                       return -ENOTCONN;
+               cfg80211_unhold_bss(wdev->current_bss);
+               cfg80211_put_bss(wdev->wiphy, &wdev->current_bss->pub);
+               wdev->current_bss = NULL;
+
+               cfg80211_sme_free(wdev);
+       }
 
        if (WARN_ON(wdev->conn))
                return -EINPROGRESS;