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_sta.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_sta.c')
-rw-r--r-- | sys/net80211/ieee80211_sta.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/net80211/ieee80211_sta.c b/sys/net80211/ieee80211_sta.c index c71f4de..743fe48 100644 --- a/sys/net80211/ieee80211_sta.c +++ b/sys/net80211/ieee80211_sta.c @@ -517,7 +517,7 @@ sta_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; @@ -542,7 +542,6 @@ sta_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)) { |