Hi Dan, Eric, thank you a lot for the finds, that were my faults.
With best regards, Alexander 2011/8/30 Eric Dumazet <eric.duma...@gmail.com>: > Le mardi 30 août 2011 à 16:45 +0300, Dan Carpenter a écrit : >> Use kfree_skb() to free sbk_buffs. >> >> Signed-off-by: Dan Carpenter <erro...@gmail.com> >> >> diff --git a/net/ieee802154/6lowpan.c b/net/ieee802154/6lowpan.c >> index cf304cc..8a9dbaa 100644 >> --- a/net/ieee802154/6lowpan.c >> +++ b/net/ieee802154/6lowpan.c >> @@ -674,7 +674,7 @@ lowpan_process_data(struct sk_buff *skb) >> sizeof(hdr)); >> return lowpan_skb_deliver(skb, &hdr); >> drop: >> - kfree(skb); >> + kfree_skb(skb); >> return -EINVAL; >> } >> > > Another bug is the skb_copy() done in lowpan_skb_deliver() > > 1) No check of skb_copy() return > > 2.1) Use of GFP_KERNEL : Is it safe at this point ? Aren’t we in > softirq ? > > 2.2) If GFP_KERNEL is safe, why do we later do : > > if (in_interrupt()) > stat = netif_rx(skb); > else > stat = netif_rx_ni(skb); > > > > ------------------------------------------------------------------------------ Special Offer -- Download ArcSight Logger for FREE! Finally, a world-class log management solution at an even better price-free! And you'll get a free "Love Thy Logs" t-shirt when you download Logger. Secure your free ArcSight Logger TODAY! http://p.sf.net/sfu/arcsisghtdev2dev _______________________________________________ Linux-zigbee-devel mailing list Linux-zigbee-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel
