diff options
| author | Felix Fietkau | 2026-02-08 18:46:07 +0000 |
|---|---|---|
| committer | Felix Fietkau | 2026-02-08 18:46:45 +0000 |
| commit | 3553eda28379f1074b8c2cc4d3ab9930552f0cad (patch) | |
| tree | 221dcc5461ba47ef02f786f2aae8399780cc1535 | |
| parent | 97b0379514c233bd48f5b9815562c7d1413831b6 (diff) | |
| download | nbd-3553eda28379f1074b8c2cc4d3ab9930552f0cad.tar.gz | |
wifi-scripts: fix spurious teardown on config_change during setup
When config_change is set during an active setup (e.g. by a concurrent
reconf call), wdev_mark_up() attempted to call setup() while still in
"setup" state. Since setup() requires state "up" or "down", it silently
returned, leaving the state as "setup". The subsequent wdev_setup_cb()
then treated this as a setup failure, triggering an unnecessary
teardown+restart cycle.
Fix this by removing the config_change handling from wdev_mark_up() and
moving it to wdev_setup_cb() instead. wdev_mark_up() now always
transitions to "up" state. When wdev_setup_cb() runs afterwards and
finds the device already "up" with config_change set, it initiates a
clean re-setup from the "up" state where setup() can run.
Signed-off-by: Felix Fietkau <nbd@nbd.name>
| -rw-r--r-- | package/network/config/wifi-scripts/files/lib/netifd/wireless-device.uc | 12 |
1 files changed, 6 insertions, 6 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 8168e291eb..768a421480 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 @@ -142,8 +142,13 @@ function wdev_config_init(wdev) function wdev_setup_cb(wdev) { - if (wdev.state != "setup") + if (wdev.state != "setup") { + if (wdev.state == "up" && wdev.config_change) { + wdev_config_init(wdev); + wdev.setup(); + } return; + } if (wdev.retry > 0) wdev.retry--; @@ -448,11 +453,6 @@ function wdev_mark_up(wdev) if (wdev.state != "setup") return; - if (wdev.config_change) { - wdev.setup(); - return; - } - for (let section, data in wdev.handler_data) { if (data.ifname) handle_link(data.ifname, data, true); |