Johannes Berg wrote:
> This code in ath9k_ht_tx looks buggy:
>
> if (padsize && skb->len > padpos) {
> if (skb_headroom(skb) < padsize)
> return -1;
> skb_push(skb, padsize);
> memmove(skb->data, skb->data + padsize, padpos);
> }
>
> -1 isn't a valid netdev tx status code -- I think this should just be
> "goto fail_tx;" instead?
Yep, I have a fix lined up in my pending queue doing exactly that. :)
Sujith
--
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
