mac80211: mesh_plink: group basic fitness checks
authorBob Copeland <me@bobcopeland.com>
Tue, 5 Nov 2013 19:16:53 +0000 (11:16 -0800)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 25 Nov 2013 19:49:58 +0000 (20:49 +0100)
The initial frame checks differ depending on whether this is
a new peer or not, but they were all intermixed with sta checks
as necessary.  Group them together so the two cases are clearer.

Signed-off-by: Bob Copeland <bob@cozybit.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/mesh_plink.c

index 3bab76f4f3b652638087f183bdc9849dc1ddb3c7..5048658709d0deedccce55c1709e606cc00cf411 100644 (file)
@@ -700,7 +700,7 @@ void mesh_rx_plink_frame(struct ieee80211_sub_if_data *sdata,
        enum plink_event event;
        enum ieee80211_self_protected_actioncode ftype;
        size_t baselen;
-       bool matches_local = true;
+       bool matches_local;
        u8 ie_len;
        u8 *baseaddr;
        u32 changed = 0;
@@ -771,11 +771,9 @@ void mesh_rx_plink_frame(struct ieee80211_sub_if_data *sdata,
        rcu_read_lock();
 
        sta = sta_info_get(sdata, mgmt->sa);
-       if (!sta && ftype != WLAN_SP_MESH_PEERING_OPEN) {
-               mpl_dbg(sdata, "Mesh plink: cls or cnf from unknown peer\n");
-               rcu_read_unlock();
-               return;
-       }
+
+       matches_local = ftype == WLAN_SP_MESH_PEERING_CLOSE ||
+                       mesh_matches_local(sdata, &elems);
 
        if (ftype == WLAN_SP_MESH_PEERING_OPEN &&
            !rssi_threshold_check(sta, sdata)) {
@@ -785,22 +783,41 @@ void mesh_rx_plink_frame(struct ieee80211_sub_if_data *sdata,
                return;
        }
 
-       if (sta && !test_sta_flag(sta, WLAN_STA_AUTH)) {
-               mpl_dbg(sdata, "Mesh plink: Action frame from non-authed peer\n");
-               rcu_read_unlock();
-               return;
-       }
-
-       if (sta && sta->plink_state == NL80211_PLINK_BLOCKED) {
-               rcu_read_unlock();
-               return;
+       if (!sta) {
+               if (ftype != WLAN_SP_MESH_PEERING_OPEN) {
+                       mpl_dbg(sdata, "Mesh plink: cls or cnf from unknown peer\n");
+                       rcu_read_unlock();
+                       return;
+               }
+               /* ftype == WLAN_SP_MESH_PEERING_OPEN */
+               if (!mesh_plink_free_count(sdata)) {
+                       mpl_dbg(sdata, "Mesh plink error: no more free plinks\n");
+                       rcu_read_unlock();
+                       return;
+               }
+               /* deny open request from non-matching peer */
+               if (!matches_local) {
+                       rcu_read_unlock();
+                       mesh_plink_frame_tx(sdata, WLAN_SP_MESH_PEERING_CLOSE,
+                                           mgmt->sa, 0, plid,
+                                           cpu_to_le16(WLAN_REASON_MESH_CONFIG));
+                       return;
+               }
+       } else {
+               if (!test_sta_flag(sta, WLAN_STA_AUTH)) {
+                       mpl_dbg(sdata, "Mesh plink: Action frame from non-authed peer\n");
+                       rcu_read_unlock();
+                       return;
+               }
+               if (sta->plink_state == NL80211_PLINK_BLOCKED) {
+                       rcu_read_unlock();
+                       return;
+               }
        }
 
        /* Now we will figure out the appropriate event... */
        event = PLINK_UNDEFINED;
-       if (ftype != WLAN_SP_MESH_PEERING_CLOSE &&
-           !mesh_matches_local(sdata, &elems)) {
-               matches_local = false;
+       if (!matches_local) {
                switch (ftype) {
                case WLAN_SP_MESH_PEERING_OPEN:
                        event = OPN_RJCT;
@@ -813,22 +830,9 @@ void mesh_rx_plink_frame(struct ieee80211_sub_if_data *sdata,
                }
        }
 
-       if (!sta && !matches_local) {
-               rcu_read_unlock();
-               llid = 0;
-               mesh_plink_frame_tx(sdata, WLAN_SP_MESH_PEERING_CLOSE,
-                                   mgmt->sa, llid, plid,
-                                   cpu_to_le16(WLAN_REASON_MESH_CONFIG));
-               return;
-       } else if (!sta) {
-               /* ftype == WLAN_SP_MESH_PEERING_OPEN */
-               if (!mesh_plink_free_count(sdata)) {
-                       mpl_dbg(sdata, "Mesh plink error: no more free plinks\n");
-                       rcu_read_unlock();
-                       return;
-               }
+       if (!sta)
                event = OPN_ACPT;
-       else if (matches_local) {
+       else if (matches_local) {
                switch (ftype) {
                case WLAN_SP_MESH_PEERING_OPEN:
                        if (!mesh_plink_free_count(sdata) ||