On Thu, 2006-05-25 at 00:13 +0200, Francois Romieu wrote:

> > +   pwol_pattern = kmalloc(B44_WOL_PATTERN_SIZE, GFP_KERNEL);
> > +   pwol_mask = kmalloc(B44_WOL_MASK_SIZE, GFP_KERNEL);
> > +
> kmalloc can fail. It should be checked.
Yep. Thanks.

> > +   max = ETH_ALEN;
> > +   for (i = 0; i < 14; ++i) {
> > +           if (i == 13)
> > +                   max = 2;
> > +           for (j = 0; j < max; ++j) {
> > +                   pwol_pattern[42 + 6 +
> > +                   (i * ETH_ALEN) + j] =
> > +                   bp->dev->dev_addr[j];
> 
> Indentation went strange here.
> 
Will adjust.
Thanks.

-
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