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_adhoc.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_adhoc.c')
| -rw-r--r-- | sys/net80211/ieee80211_adhoc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/net80211/ieee80211_adhoc.c b/sys/net80211/ieee80211_adhoc.c index 8055890..d3b43bd 100644 --- a/sys/net80211/ieee80211_adhoc.c +++ b/sys/net80211/ieee80211_adhoc.c @@ -293,7 +293,7 @@ adhoc_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; @@ -318,7 +318,6 @@ adhoc_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)) { |
