On Fri, 2012-11-23 at 08:25 +0100, Wojciech Dubowik wrote: > On 11/22/2012 09:40 AM, Wojciech Dubowik wrote: > > On 11/22/2012 09:41 AM, Johannes Berg wrote: > >> On Thu, 2012-11-22 at 09:34 +0100, Johannes Berg wrote: > >>> On Thu, 2012-11-22 at 09:22 +0100, Wojciech Dubowik wrote: > >>>> Shouldn't all functions between ieee80211_rx > >>>> and remove_monitor_info map header with > >>>> (struct ieee80211_hdr *)(skb->data + status->vendor_radiotap_len) ? > >>>> > >>>> Otherwise function like should_drop_frame uses header mapped over > >>>> potential vendor data. > >>> Huh, yes, how did I miss that. It only applies to should_drop_frame() > >>> though, or do you see any other place? > >> So I think this is sufficient? > > I guess so. I will test it today. > > Wojtek > I have tested with ath9k vendor data and it doesn't seem to drop frames. > I will post my patch as RFC > once I clean it up a bit. Just for reference. Great, thanks. I'll send out the fix & merge it. 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
