diff options
| author | Felix Fietkau <nbd@openwrt.org> | 2015-04-11 10:51:55 +0000 |
|---|---|---|
| committer | Felix Fietkau <nbd@openwrt.org> | 2015-04-11 10:51:55 +0000 |
| commit | daaed6f251c7714cdf7aec6dc2940a605e82eaed (patch) | |
| tree | 4be03f0b408506783e9d5fd1005ced40753d1afd /package | |
| parent | bb828e30b04a309b700f8e852a5bee6b4cb97b0b (diff) | |
| download | master-31e0f0ae-daaed6f251c7714cdf7aec6dc2940a605e82eaed.tar.gz master-31e0f0ae-daaed6f251c7714cdf7aec6dc2940a605e82eaed.tar.bz2 master-31e0f0ae-daaed6f251c7714cdf7aec6dc2940a605e82eaed.zip | |
netifd: fix start_disabled 'sh: bad number' in mac80211 setup (bug #19345)
Two errors "netifd: radio0: sh: bad number" have recently surfaced in system
log in trunk when wifi interfaces come up. I tracked the errors to checking
numerical values of some config options without ensuring that the option has
any value.
The errors I see have apparently been introduced by r45051 (ieee80211r in
hostapd) and r45326 (start_disabled in mac80211). My patches fix two
instances of "bad number", but there may be a third one, as the original
report in bug 19345 pre-dates r45326 and already has two "bad number" errors
for radio0.
https://dev.openwrt.org/ticket/19345
Signed-off-by: Hannu Nyman <hannu.nyman@iki.fi>
SVN-Revision: 45379
Diffstat (limited to 'package')
| -rw-r--r-- | package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh b/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh index a7848bd8ad..ec0966ea8d 100644 --- a/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh +++ b/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh @@ -318,6 +318,7 @@ mac80211_hostapd_setup_bss() { json_get_vars wds dtim_period max_listen_int start_disabled set_default wds 0 + set_default start_disabled 0 [ "$wds" -gt 0 ] && append hostapd_cfg "wds_sta=1" "$N" [ "$staidx" -gt 0 -o "$start_disabled" -eq 1 ] && append hostapd_cfg "start_disabled=1" "$N" |