PERFORCE change 135971 for review
Sam Leffler
sam at FreeBSD.org
Fri Feb 22 17:00:47 UTC 2008
http://perforce.freebsd.org/chv.cgi?CH=135971
Change 135971 by sam at sam_ebb on 2008/02/22 17:00:03
pull check that SA is unicast to common mgt frame processing so
all frames are checked
Affected files ...
.. //depot/projects/vap/sys/net80211/ieee80211_hostap.c#15 edit
Differences ...
==== //depot/projects/vap/sys/net80211/ieee80211_hostap.c#15 (text+ko) ====
@@ -811,6 +811,14 @@
vap->iv_stats.is_rx_tooshort++;
goto out;
}
+ if (IEEE80211_IS_MULTICAST(wh->i_addr2)) {
+ /* ensure return frames are unicast */
+ IEEE80211_DISCARD(vap, IEEE80211_MSG_ANY,
+ wh, NULL, "source is multicast: %s",
+ ether_sprintf(wh->i_addr2));
+ vap->iv_stats.is_rx_mgtdiscard++; /* XXX stat */
+ goto out;
+ }
#ifdef IEEE80211_DEBUG
if ((ieee80211_msg_debug(vap) && doprint(vap, subtype)) ||
ieee80211_msg_dumppkts(vap)) {
@@ -1742,11 +1750,6 @@
vap->iv_stats.is_rx_mgtdiscard++;
return;
}
- if (IEEE80211_IS_MULTICAST(wh->i_addr2)) {
- /* frame must be directed */
- vap->iv_stats.is_rx_mgtdiscard++; /* XXX stat */
- return;
- }
/*
* prreq frame format
More information about the p4-projects
mailing list