diff options
author | sam <sam@FreeBSD.org> | 2009-05-25 16:38:47 +0000 |
---|---|---|
committer | sam <sam@FreeBSD.org> | 2009-05-25 16:38:47 +0000 |
commit | c16474afa051485a6b6bb1b5228e59f1e8b30927 (patch) | |
tree | 2e0eaf018173b7a9aac32f2e0adcc48f0bf3bc7c /sys/net80211/ieee80211_radiotap.c | |
parent | 1dcc1a56a6b94b2719df947054688a1478e503cb (diff) | |
download | FreeBSD-src-c16474afa051485a6b6bb1b5228e59f1e8b30927.zip FreeBSD-src-c16474afa051485a6b6bb1b5228e59f1e8b30927.tar.gz |
Fix handling of devices w/o radiotap support:
o do not attach DLT_IEEE802_11_RADIO unless both tx and rx headers are
present; this is assumed in the capture code paths
o verify the above with asserts in ieee80211_radiotap_{rx,tx}
o add missing checks for active taps before calling ieee80211_radiotap_rx
Diffstat (limited to 'sys/net80211/ieee80211_radiotap.c')
-rw-r--r-- | sys/net80211/ieee80211_radiotap.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/sys/net80211/ieee80211_radiotap.c b/sys/net80211/ieee80211_radiotap.c index 2c4482f..9c8dc4d 100644 --- a/sys/net80211/ieee80211_radiotap.c +++ b/sys/net80211/ieee80211_radiotap.c @@ -102,12 +102,12 @@ ieee80211_radiotap_vattach(struct ieee80211vap *vap) struct ieee80211com *ic = vap->iv_ic; struct ieee80211_radiotap_header *th = ic->ic_th; - KASSERT(th != NULL, ("no radiotap setup")); - - /* radiotap DLT for raw 802.11 frames */ - bpfattach2(vap->iv_ifp, DLT_IEEE802_11_RADIO, - sizeof(struct ieee80211_frame) + le16toh(th->it_len), - &vap->iv_rawbpf); + if (th != NULL && ic->ic_rh != NULL) { + /* radiotap DLT for raw 802.11 frames */ + bpfattach2(vap->iv_ifp, DLT_IEEE802_11_RADIO, + sizeof(struct ieee80211_frame) + le16toh(th->it_len), + &vap->iv_rawbpf); + } } void @@ -193,6 +193,7 @@ dispatch_radiotap(struct ieee80211vap *vap0, struct mbuf *m, void ieee80211_radiotap_tx(struct ieee80211vap *vap0, struct mbuf *m) { + KASSERT(vap0->iv_ic->ic_th != NULL, ("no tx radiotap header")); dispatch_radiotap(vap0, m, vap0->iv_ic->ic_th); } @@ -202,6 +203,7 @@ ieee80211_radiotap_tx(struct ieee80211vap *vap0, struct mbuf *m) void ieee80211_radiotap_rx(struct ieee80211vap *vap0, struct mbuf *m) { + KASSERT(vap0->iv_ic->ic_rh != NULL, ("no rx radiotap header")); dispatch_radiotap(vap0, m, vap0->iv_ic->ic_rh); } |