diff options
author | Stefan Sperling <stsp@cvs.openbsd.org> | 2016-01-04 12:32:07 +0000 |
---|---|---|
committer | Stefan Sperling <stsp@cvs.openbsd.org> | 2016-01-04 12:32:07 +0000 |
commit | 7bc2740aa9d97b4ec12871c36f57fa5d52335204 (patch) | |
tree | 602eb45c3dd8b5f3a0273a1a4ed3e79fa8392905 /sys/net80211/ieee80211_input.c | |
parent | 111f7030937b615e7b4931462886449db5d3552e (diff) |
ADDBA frames have a parameter set which we check against our own capabilities
but we were checking bits in these parameters with the wrong set of bitmasks.
Negotiating A-MPDUs with some APs failed because of this bug.
ok kettenis@
Diffstat (limited to 'sys/net80211/ieee80211_input.c')
-rw-r--r-- | sys/net80211/ieee80211_input.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net80211/ieee80211_input.c b/sys/net80211/ieee80211_input.c index 22e532ef99a..4aecf1667d9 100644 --- a/sys/net80211/ieee80211_input.c +++ b/sys/net80211/ieee80211_input.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ieee80211_input.c,v 1.147 2016/01/04 12:25:46 stsp Exp $ */ +/* $OpenBSD: ieee80211_input.c,v 1.148 2016/01/04 12:32:06 stsp Exp $ */ /*- * Copyright (c) 2001 Atsushi Onoe @@ -2459,7 +2459,7 @@ ieee80211_recv_addba_req(struct ieee80211com *ic, struct mbuf *m, } /* check that we support the requested Block Ack Policy */ if (!(ic->ic_htcaps & IEEE80211_HTCAP_DELAYEDBA) && - !(params & IEEE80211_BA_ACK_POLICY)) { + !(params & IEEE80211_ADDBA_BA_POLICY)) { status = IEEE80211_STATUS_INVALID_PARAM; goto resp; } |