On Mon, 2011-05-09 at 17:19 +0300, Luciano Coelho wrote: > This seems to be a leftover from the old days, when we didn't support > any frames that didn't contain the full ieee802.11 header. This is > not the case anymore. It does not cause problems now, because they > are only dropped during scan. But when scheduled scans get merged, > this would become a problem because we would drop all small frames > while scheduled scan is running. > > Cc: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > Signed-off-by: Luciano Coelho <coelho@xxxxxx> > --- > net/mac80211/scan.c | 3 --- > 1 files changed, 0 insertions(+), 3 deletions(-) > > diff --git a/net/mac80211/scan.c b/net/mac80211/scan.c > index 489b6ad..b7a39c4 100644 > --- a/net/mac80211/scan.c > +++ b/net/mac80211/scan.c > @@ -169,9 +169,6 @@ ieee80211_scan_rx(struct ieee80211_sub_if_data *sdata, struct sk_buff *skb) > if (ieee80211_is_ctl(fc)) > return RX_CONTINUE; > > - if (skb->len < 24) > - return RX_DROP_MONITOR; > - You can't just delete it, we use mgmt->da at least in this code path. You should just change RX_DROP_MONITOR to RX_CONTINUe here I think. johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
