diff options
| author | Felix Fietkau | 2026-02-04 10:34:55 +0000 |
|---|---|---|
| committer | Felix Fietkau | 2026-02-07 09:23:22 +0000 |
| commit | 68c2ab8f5fdf1c12966fa0cdf3c13d889beccdbe (patch) | |
| tree | abeea050687140d49b8676c2c6d11f0aee022d20 | |
| parent | 6e25c8bd785e72b8a03b1f1f6bacc21d077cc372 (diff) | |
| download | nbd-68c2ab8f5fdf1c12966fa0cdf3c13d889beccdbe.tar.gz | |
wifi-scripts: fix nested config accumulation in wdev_set_data
When storing device-level data, wdev_set_data() spread the entire wdev
object into handler_data. Since handler_config.data is set from the
previous handler_data[wdev.name] before each setup, this created
exponentially growing nesting with each reload, eventually causing
"nesting too deep" JSON parse errors.
Fix by initializing cur to a simple object containing only the device
name instead of the entire wdev object.
Signed-off-by: Felix Fietkau <nbd@nbd.name>
| -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 428db6ee6a..8168e291eb 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 @@ -465,7 +465,7 @@ function wdev_mark_up(wdev) function wdev_set_data(wdev, vif, vlan, data) { let config = wdev.handler_config; - let cur = wdev; + let cur = { name: wdev.name }; let cur_type = "device"; if (!config) return ubus.STATUS_INVALID_ARGUMENT; |