On Wed, Dec 10, 2008 at 8:24 AM, Bob Copeland <me@xxxxxxxxxxxxxxx> wrote:
> On Wed, Dec 10, 2008 at 06:04:05AM +0100, Patrick McHardy wrote:
>> This might not be fully correct or not handle other cases where
>> this can occur, but it doesn't seem too hackish and fixes the
>> problem for me :)
>
>> - if (hdrlen & 3) {
>> + if (hdrlen & 3 && hdrlen != rs.rs_datalen - FCS_LEN) {
>> pad = hdrlen % 4;
>> memmove(skb->data + pad, skb->data, hdrlen);
>> skb_pull(skb, pad);
>
> It seems very plausible to me. Though, why doesn't ath9k also have this
> problem?
So, I guess ath9k did too. This patch gets my ack, though we should probably
do the hdrlen check the same way as Jouni's patch for consistency, and switch
to hdrlen & 3 when computing pad.
--
Bob Copeland %% www.bobcopeland.com
--
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
