From bd8dad29fa8ed36a2518bec8f95a8fe855517df7 Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Fri, 14 Nov 2008 23:35:12 +0000 Subject: [PATCH] madwifi: ap+sta race conditions SVN-Revision: 13208 --- package/madwifi/files/lib/wifi/madwifi.sh | 2 +- package/madwifi/patches/388-apsta_fix.patch | 60 +++++++++++++++++++++ 2 files changed, 61 insertions(+), 1 deletion(-) create mode 100644 package/madwifi/patches/388-apsta_fix.patch diff --git a/package/madwifi/files/lib/wifi/madwifi.sh b/package/madwifi/files/lib/wifi/madwifi.sh index 5ce288f6e6..0ff9292600 100755 --- a/package/madwifi/files/lib/wifi/madwifi.sh +++ b/package/madwifi/files/lib/wifi/madwifi.sh @@ -42,7 +42,7 @@ scan_atheros() { *) echo "$device: Invalid mode combination in config"; return 1;; esac - config_set "$device" vifs "${ap:+$ap }${adhoc:+$adhoc }${ahdemo:+$ahdemo }${sta:+$sta }${wds:+$wds }${monitor:+$monitor}" + config_set "$device" vifs "${sta:+$sta }${ap:+$ap }${adhoc:+$adhoc }${ahdemo:+$ahdemo }${wds:+$wds }${monitor:+$monitor}" } diff --git a/package/madwifi/patches/388-apsta_fix.patch b/package/madwifi/patches/388-apsta_fix.patch new file mode 100644 index 0000000000..4952dd8913 --- /dev/null +++ b/package/madwifi/patches/388-apsta_fix.patch @@ -0,0 +1,60 @@ +--- a/net80211/ieee80211_proto.c ++++ b/net80211/ieee80211_proto.c +@@ -1406,7 +1406,8 @@ __ieee80211_newstate(struct ieee80211vap + vap->iv_state = nstate; /* state transition */ + del_timer(&vap->iv_mgtsend); + if ((vap->iv_opmode != IEEE80211_M_HOSTAP) && +- (ostate != IEEE80211_S_SCAN)) ++ (ostate != IEEE80211_S_SCAN) && ++ !(vap->iv_flags_ext & IEEE80211_FEXT_SCAN_PENDING)) + ieee80211_cancel_scan(vap); /* background scan */ + ni = vap->iv_bss; /* NB: no reference held */ + switch (nstate) { +@@ -1448,7 +1449,8 @@ __ieee80211_newstate(struct ieee80211vap + } + goto reset; + case IEEE80211_S_SCAN: +- ieee80211_cancel_scan(vap); ++ if (!(vap->iv_flags_ext & IEEE80211_FEXT_SCAN_PENDING)) ++ ieee80211_cancel_scan(vap); + goto reset; + reset: + ieee80211_reset_bss(vap); +@@ -1985,7 +1987,9 @@ ieee80211_newstate(struct ieee80211vap * + } + } + } +- } else if (dstate == IEEE80211_S_SCAN) { ++ } else if ((dstate == IEEE80211_S_SCAN) || ++ (dstate == IEEE80211_S_AUTH) || ++ (dstate == IEEE80211_S_ASSOC)) { + /* Force to scan pending... someone is scanning */ + vap->iv_flags_ext |= IEEE80211_FEXT_SCAN_PENDING; + __ieee80211_newstate(vap, IEEE80211_S_INIT, arg); +--- a/net80211/ieee80211_output.c ++++ b/net80211/ieee80211_output.c +@@ -238,7 +238,9 @@ ieee80211_hardstart(struct sk_buff *skb, + } + + /* Cancel any running BG scan */ +- if (!(ic->ic_flags_ext & IEEE80211_FEXT_BGSCAN_THR) && (vap->iv_state == IEEE80211_S_RUN)) ++ if (!(ic->ic_flags_ext & IEEE80211_FEXT_BGSCAN_THR) && ++ (vap->iv_state == IEEE80211_S_RUN) && ++ (ic->ic_flags_ext & IEEE80211_FEXT_BGSCAN)) + ieee80211_cancel_scan(vap); + + /* +--- a/net80211/ieee80211_wireless.c ++++ b/net80211/ieee80211_wireless.c +@@ -2728,9 +2728,9 @@ ieee80211_ioctl_setparam(struct net_devi + return -EINVAL; + vap->iv_flags |= IEEE80211_F_BGSCAN; + } else { +- /* XXX racey? */ ++ if (ic->ic_flags_ext & IEEE80211_FEXT_BGSCAN) ++ ieee80211_cancel_scan(vap); /* anything current */ + vap->iv_flags &= ~IEEE80211_F_BGSCAN; +- ieee80211_cancel_scan(vap); /* anything current */ + } + break; + case IEEE80211_PARAM_BGSCAN_IDLE: -- 2.30.2