Dan Carpenter
2012-Jun-30 11:48 UTC
[Bridge] [patch -next] netfilter: use kfree_skb() not kfree()
This was should be a kfree_skb() here to free the sk_buff pointer. Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com> diff --git a/net/bridge/netfilter/ebt_ulog.c b/net/bridge/netfilter/ebt_ulog.c index 1bd1732..dfbb019 100644 --- a/net/bridge/netfilter/ebt_ulog.c +++ b/net/bridge/netfilter/ebt_ulog.c @@ -156,7 +156,7 @@ static void ebt_ulog_packet(unsigned int hooknr, const struct sk_buff *skb, nlh = nlmsg_put(ub->skb, 0, ub->qlen, 0, size - NLMSG_ALIGN(sizeof(*nlh)), 0); if (!nlh) { - kfree(ub->skb); + kfree_skb(ub->skb); ub->skb = NULL; goto unlock; }
David Miller
2012-Jul-01 00:27 UTC
[Bridge] [patch -next] netfilter: use kfree_skb() not kfree()
From: Dan Carpenter <dan.carpenter at oracle.com> Date: Sat, 30 Jun 2012 14:48:53 +0300> This was should be a kfree_skb() here to free the sk_buff pointer. > > Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>My bad, applied, thanks Dan.