Brandon Craig Rhodes wrote:
> +     if (skb_tailroom(skb) < 4) {
> +             int err;
> +             err = skb_padto(skb, skb->len + 4);
> +             if (unlikely(err || skb_tailroom(skb) < 4)) {
> +                     printk(KERN_DEBUG "Failed to increase tailroom"
> +                            " for TKIP encrypt");
> +                     return err || -1;

The "||" operator in C doesn't act the same way it does in perl and ruby.
You're always returning 1 here.

+                       return err || -1;

...and here as well.

-Mitch
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to