staging: wfx: fix BA when device is AP and MFP is enabled
authorJérôme Pouiller <jerome.pouiller@silabs.com>
Tue, 25 Aug 2020 08:58:17 +0000 (10:58 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 28 Aug 2020 10:19:50 +0000 (12:19 +0200)
The protection of the management frames is mainly done by mac80211.
However, frames for the management of the BlockAck sessions are directly
sent by the device. These frames have to be protected if MFP is in use.
So the driver has to pass the MFP configuration to the device.

Until now, the BlockAck management frames were completely unprotected
whatever the status of the MFP negotiation. So, some devices dropped
these frames.

The device has two knobs to control the MFP. One global and one per
station. Normally, the driver should always enable global MFP. Then it
should enable MFP on every station with which MFP was successfully
negotiated. Unfortunately, the older firmwares only provide the
global control.

So, this patch enable global MFP as it is exposed in the beacon. Then it
marks every station with which the MFP is effective.

Thus, the support for the old firmwares is not so bad. It may only
encounter some difficulties to negotiate BA sessions when the local
device (the AP) is MFP capable (ieee80211w=1) but the station is not.
The only solution for this case is to upgrade the firmware.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200825085828.399505-1-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wfx/sta.c

index ad63332..c31e302 100644 (file)
@@ -434,7 +434,7 @@ int wfx_sta_add(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
        wvif->link_id_map |= BIT(sta_priv->link_id);
        WARN_ON(!sta_priv->link_id);
        WARN_ON(sta_priv->link_id >= HIF_LINK_ID_MAX);
-       hif_map_link(wvif, sta->addr, 0, sta_priv->link_id);
+       hif_map_link(wvif, sta->addr, sta->mfp ? 2 : 0, sta_priv->link_id);
 
        return 0;
 }
@@ -474,6 +474,31 @@ static int wfx_upload_ap_templates(struct wfx_vif *wvif)
        return 0;
 }
 
+static void wfx_set_mfp_ap(struct wfx_vif *wvif)
+{
+       struct sk_buff *skb = ieee80211_beacon_get(wvif->wdev->hw, wvif->vif);
+       const int ieoffset = offsetof(struct ieee80211_mgmt, u.beacon.variable);
+       const u16 *ptr = (u16 *)cfg80211_find_ie(WLAN_EID_RSN,
+                                                skb->data + ieoffset,
+                                                skb->len - ieoffset);
+       const int pairwise_cipher_suite_count_offset = 8 / sizeof(u16);
+       const int pairwise_cipher_suite_size = 4 / sizeof(u16);
+       const int akm_suite_size = 4 / sizeof(u16);
+
+       if (ptr) {
+               ptr += pairwise_cipher_suite_count_offset;
+               if (WARN_ON(ptr > (u16 *)skb_tail_pointer(skb)))
+                       return;
+               ptr += 1 + pairwise_cipher_suite_size * *ptr;
+               if (WARN_ON(ptr > (u16 *)skb_tail_pointer(skb)))
+                       return;
+               ptr += 1 + akm_suite_size * *ptr;
+               if (WARN_ON(ptr > (u16 *)skb_tail_pointer(skb)))
+                       return;
+               hif_set_mfp(wvif, *ptr & BIT(7), *ptr & BIT(6));
+       }
+}
+
 int wfx_start_ap(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
 {
        struct wfx_vif *wvif = (struct wfx_vif *)vif->drv_priv;
@@ -488,6 +513,7 @@ int wfx_start_ap(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
        ret = hif_start(wvif, &vif->bss_conf, wvif->channel);
        if (ret > 0)
                return -EIO;
+       wfx_set_mfp_ap(wvif);
        return ret;
 }