MBSSID: Check xrates_supported for all BSSs explicitly

This is needed to avoid generating an nontransmitted BSS profile that
would claim the Extended Rates element to be non-inherited.

Signed-off-by: Jouni Malinen <quic_jouni@quicinc.com>
This commit is contained in:
Jouni Malinen 2023-10-31 15:52:42 +02:00 committed by Jouni Malinen
parent 4bfc007b61
commit ab3e679ae5

View file

@ -471,6 +471,7 @@ ieee802_11_build_ap_params_mbssid(struct hostapd_data *hapd,
size_t len, rnr_len = 0; size_t len, rnr_len = 0;
u8 elem_count = 0, *elem = NULL, **elem_offset = NULL, *end; u8 elem_count = 0, *elem = NULL, **elem_offset = NULL, *end;
u8 rnr_elem_count = 0, *rnr_elem = NULL, **rnr_elem_offset = NULL; u8 rnr_elem_count = 0, *rnr_elem = NULL, **rnr_elem_offset = NULL;
size_t i;
if (!iface->mbssid_max_interfaces || if (!iface->mbssid_max_interfaces ||
iface->num_bss > iface->mbssid_max_interfaces || iface->num_bss > iface->mbssid_max_interfaces ||
@ -478,6 +479,14 @@ ieee802_11_build_ap_params_mbssid(struct hostapd_data *hapd,
!iface->ema_max_periodicity)) !iface->ema_max_periodicity))
goto fail; goto fail;
/* Make sure bss->xrates_supported is set for all BSSs to know whether
* it need to be non-inherited. */
for (i = 0; i < iface->num_bss; i++) {
u8 buf[100];
hostapd_eid_ext_supp_rates(iface->bss[i], buf);
}
tx_bss = hostapd_mbssid_get_tx_bss(hapd); tx_bss = hostapd_mbssid_get_tx_bss(hapd);
len = hostapd_eid_mbssid_len(tx_bss, WLAN_FC_STYPE_BEACON, &elem_count, len = hostapd_eid_mbssid_len(tx_bss, WLAN_FC_STYPE_BEACON, &elem_count,
NULL, 0, &rnr_len); NULL, 0, &rnr_len);