On Thu, 2012-10-25 at 11:10 -0700, Javier Cardona wrote: > Per IEEE Std. 802.11-2012, Sec 8.2.4.4.1, the sequence Control field is > not present in control frames. We noticed this problem when processing > Block Ack Requests. > Cc stable? > hdr = (struct ieee80211_hdr *)rx->skb->data; > fc = hdr->frame_control; > + > + if (ieee80211_is_ctl(fc)) > + return RX_CONTINUE; Shouldn't that be "goto out"? And it seems it should also incorporate the skb->len check here, rather than accessing the field before checking that it's present? 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