diff options
| author | Stefan Sperling <stsp@cvs.openbsd.org> | 2021-12-03 12:40:16 +0000 |
|---|---|---|
| committer | Stefan Sperling <stsp@cvs.openbsd.org> | 2021-12-03 12:40:16 +0000 |
| commit | 63c341751462aadfbefca44bba02aa90f9070f7f (patch) | |
| tree | 074c18c0e92518ed4401ffc080610956f52e5ee5 | |
| parent | 067cbc8581ca0c9298a8dac5e48f33cb0ece2a23 (diff) | |
Ignore ADDBA requests from our AP while we are roaming away from it.
Noticed while testing iwm/iwx roaming patches, where my AP would request
a new Rx BA session when we had already decided to roam away. There is no
need to set up a new Rx BA session with our old AP which we would have to
immediately tear down again anyway.
| -rw-r--r-- | sys/net80211/ieee80211_input.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/net80211/ieee80211_input.c b/sys/net80211/ieee80211_input.c index 888a39c43dd..bb6367fd1d5 100644 --- a/sys/net80211/ieee80211_input.c +++ b/sys/net80211/ieee80211_input.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ieee80211_input.c,v 1.239 2021/10/11 09:02:01 stsp Exp $ */ +/* $OpenBSD: ieee80211_input.c,v 1.240 2021/12/03 12:40:15 stsp Exp $ */ /*- * Copyright (c) 2001 Atsushi Onoe @@ -2822,6 +2822,10 @@ ieee80211_recv_addba_req(struct ieee80211com *ic, struct mbuf *m, /* The driver is still processing an ADDBA request for this tid. */ if (ba->ba_state == IEEE80211_BA_REQUESTED) return; + /* If we are in the process of roaming between APs, ignore. */ + if ((ic->ic_flags & IEEE80211_F_BGSCAN) && + (ic->ic_xflags & IEEE80211_F_TX_MGMT_ONLY)) + return; /* check if we already have a Block Ack agreement for this RA/TID */ if (ba->ba_state == IEEE80211_BA_AGREED) { /* XXX should we update the timeout value? */ |