On Fri, 2013-09-13 at 16:36 -0700, Chun-Yeow Yeoh wrote: > + skb = dev_alloc_skb(local->tx_headroom + hdr_len + > + 5 + /* channel switch announcement element */ > + 3 + /* secondary channel offset element */ > + 8); /* mesh channel switch parameters element */ > + if (!skb) > + return -1; please fix that to -ENOMEM while at it > +} > + Please no blank line at EOF 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