nl80211: Remove redundant put_freq call in set_ap() for AP MLD

wpa_driver_nl80211_set_ap() called nl80211_put_freq_params() twice if AP
is an AP MLD. It called once while putting the MLO link ID and the other
time in the normal flow if frequency info is present. Doing this twice
is not required.

Call put_freq once during the normal flow only and separately of that,
add the link ID for AP MLD.

Signed-off-by: Aditya Kumar Singh <quic_adisi@quicinc.com>
This commit is contained in:
Aditya Kumar Singh 2024-03-06 12:08:40 +05:30 committed by Jouni Malinen
parent 4200657338
commit b810426eaa

View file

@ -5104,9 +5104,7 @@ static int wpa_driver_nl80211_set_ap(void *priv,
params->mld_link_id); params->mld_link_id);
if (nla_put_u8(msg, NL80211_ATTR_MLO_LINK_ID, if (nla_put_u8(msg, NL80211_ATTR_MLO_LINK_ID,
params->mld_link_id) || params->mld_link_id))
(params->freq &&
nl80211_put_freq_params(msg, params->freq) < 0))
goto fail; goto fail;
if (params->freq) if (params->freq)