diff options
| author | Felix Fietkau | 2026-06-26 17:00:36 +0000 |
|---|---|---|
| committer | Felix Fietkau | 2026-06-26 17:02:25 +0000 |
| commit | b830633cb82ea6c07de76395679c5db8c3bf260e (patch) | |
| tree | 2cec600ddc78706e9022bae87ab349b00c9c5578 | |
| parent | eb8cd69f9121196a3c43acdf9af3c372a4a50c93 (diff) | |
| download | openwrt-b830633cb82ea6c07de76395679c5db8c3bf260e.tar.gz | |
wifi-scripts: fix disabled vif tracking using wrong dictionary key
wdev_update_disabled_vifs() indexed the disabled_vifs map with the wdev
object instead of the vif name. The map is read back via
disabled_vifs[vif.name], so disabled vifs were never matched, and
coercing the wdev object into a string key triggered an infinite
escaping loop in ucv_to_string_json_encoded() during state output,
causing OOM on reboot.
Suggested-by: Shayne Chen (陳軒丞) <Shayne.Chen@mediatek.com>
Reported-by: Michael-cy Lee (李峻宇) <Michael-cy.Lee@mediatek.com>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
(cherry picked from commit 7dd477918368b013f241ae9e0b0bda15f2a0f83a)
| -rw-r--r-- | package/network/config/wifi-scripts/files/lib/netifd/wireless-device.uc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/package/network/config/wifi-scripts/files/lib/netifd/wireless-device.uc b/package/network/config/wifi-scripts/files/lib/netifd/wireless-device.uc index 768a421480..f55fc31189 100644 --- a/package/network/config/wifi-scripts/files/lib/netifd/wireless-device.uc +++ b/package/network/config/wifi-scripts/files/lib/netifd/wireless-device.uc @@ -353,7 +353,7 @@ function wdev_update_disabled_vifs(wdev) let name = vif.name; if (enabled == false) - disabled[wdev] = true; + disabled[name] = true; else if (ifindex != cache[name]) changed = true; |