diff options
author | rpaulo <rpaulo@FreeBSD.org> | 2010-02-03 10:07:43 +0000 |
---|---|---|
committer | rpaulo <rpaulo@FreeBSD.org> | 2010-02-03 10:07:43 +0000 |
commit | adb938921e527a609b1972aa726b47ea6ca0b678 (patch) | |
tree | 318588ebc25de88382d3ff79feab42e24003fc7d /sys/net80211/ieee80211_hostap.c | |
parent | 45cc6dda971b609844e44a7644cf6d9b3f86d078 (diff) | |
download | FreeBSD-src-adb938921e527a609b1972aa726b47ea6ca0b678.zip FreeBSD-src-adb938921e527a609b1972aa726b47ea6ca0b678.tar.gz |
When taking the AMPDU reorder fastpath, need_tap wasn't being
initialized. Initialize on declaration to avoid this.
Found with: clang static analyzer
Diffstat (limited to 'sys/net80211/ieee80211_hostap.c')
-rw-r--r-- | sys/net80211/ieee80211_hostap.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/net80211/ieee80211_hostap.c b/sys/net80211/ieee80211_hostap.c index 6083cf5..8405b4f 100644 --- a/sys/net80211/ieee80211_hostap.c +++ b/sys/net80211/ieee80211_hostap.c @@ -480,7 +480,7 @@ hostap_input(struct ieee80211_node *ni, struct mbuf *m, int rssi, int nf) struct ieee80211_frame *wh; struct ieee80211_key *key; struct ether_header *eh; - int hdrspace, need_tap; + int hdrspace, need_tap = 1; /* mbuf need to be tapped. */ uint8_t dir, type, subtype, qos; uint8_t *bssid; uint16_t rxseq; @@ -505,7 +505,6 @@ hostap_input(struct ieee80211_node *ni, struct mbuf *m, int rssi, int nf) KASSERT(ni != NULL, ("null node")); ni->ni_inact = ni->ni_inact_reload; - need_tap = 1; /* mbuf need to be tapped. */ type = -1; /* undefined */ if (m->m_pkthdr.len < sizeof(struct ieee80211_frame_min)) { |