On Fri, 2021-08-20 at 08:20 -0400, Wen Gong wrote:
>
> if (!(ifmgd->flags & IEEE80211_STA_DISABLE_HE)) {
> const struct cfg80211_bss_ies *ies;
> + struct ieee80211_bss_conf *bss_conf;
am I missing where you set this?
> + if (is_6ghz) {
> + struct ieee802_11_elems elems;
This is pretty big, not sure we want it on the stack (causes warnings
for me in build). Also, if we're doing this anyway, then we can change
the code above (perhaps as a separate patch) to not do
cfg80211_find_ext_ie() but rather take it out of the parsed.
> + u8 i, j = 0;
> +
> + ieee802_11_parse_elems(ies->data, ies->len, false, &elems,
(line too long)
> + NULL, NULL);
> +
> + if (elems.pwr_constr_elem)
> + bss_conf->pwr_reduction = *elems.pwr_constr_elem;
before using it?
> +
> + BUILD_BUG_ON(ARRAY_SIZE(bss_conf->tx_pwr_env) !=
> + ARRAY_SIZE(elems.tx_pwr_env));
> +
> + for (i = 0; i < elems.tx_pwr_env_num; i++) {
> + if (elems.tx_pwr_env_len[i] >
> + sizeof(bss_conf->tx_pwr_env[j]))
> + continue;
I did that in the parsing itself.
> +
> + bss_conf->tx_pwr_env_num++;
> + memcpy(&bss_conf->tx_pwr_env[j], elems.tx_pwr_env[i],
> + elems.tx_pwr_env_len[i]);
You're never resetting any of this for the next connection (if it's not
6 GHz for example, or doesn't have any data) - should probably memset
all the new members to 0 before the "if (is_6ghz)" or so?
johannes
