gitops/mt76
1
0
Fork 0
mirror of https://github.com/outbackdingo/mt76.git synced 2026-10-24 13:26:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

wifi: mt76: mt7925: publish msta->link after successful link add

Browse Source 
Move the msta->link[link_id] publication until after
mt7925_mac_link_sta_add() succeeds.

msta->link[] is RCU-visible, so publishing it before setup completes can
expose a link whose add path later fails. Publish it only after success
to avoid partially initialized link state becoming visible.

Signed-off-by: Sean Wang <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Felix Fietkau <[email protected]>
This commit is contained in:
Sean Wang
2026-03-14 18:10:20 +00:00
committed by Felix Fietkau
parent 9fbe29ba42
commit 8be7277306
1 changed files with 14 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mt7925/main.c
+14 -4
View File
@@ -1000,10 +1000,11 @@ mt7925_mac_sta_add_links(struct mt792x_dev *dev, struct ieee80211_vif *vif,
for_each_set_bit(link_id, &new_links, IEEE80211_MLD_MAX_NUM_LINKS) {
struct ieee80211_link_sta *link_sta;
struct mt792x_link_sta *mlink;
bool is_deflink = false;
if (msta->deflink_id == IEEE80211_LINK_UNSPECIFIED) {
mlink = &msta->deflink;
msta->deflink_id = link_id;
is_deflink = true;
} else {
mlink = kzalloc(sizeof(*mlink), GFP_KERNEL);
if (!mlink) {
@@ -1012,14 +1013,23 @@ mt7925_mac_sta_add_links(struct mt792x_dev *dev, struct ieee80211_vif *vif,
}
}
msta->valid_links |= BIT(link_id);
rcu_assign_pointer(msta->link[link_id], mlink);
mlink->sta = msta;
mlink->pri_link = &sta->deflink;
mlink->wcid.def_wcid = &msta->deflink.wcid;
link_sta = mt792x_sta_to_link_sta(vif, sta, link_id);
mt7925_mac_link_sta_add(&dev->mt76, vif, link_sta, mlink);
err = mt7925_mac_link_sta_add(&dev->mt76, vif, link_sta, mlink);
if (err) {
if (!is_deflink)
kfree_rcu(mlink, rcu_head);
break;
}
if (is_deflink)
msta->deflink_id = link_id;
rcu_assign_pointer(msta->link[link_id], mlink);
msta->valid_links |= BIT(link_id);
}
return err;