netfilter: use kfree_skb() not kfree()

Dan Carpenter · davem330 · commit f7eadafb13da · 2012-06-30T17:26:51.000-07:00
This was should be a kfree_skb() here to free the sk_buff pointer.

Signed-off-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git 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;
 	}

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ static void ebt_ulog_packet(unsigned int hooknr, const struct sk_buff *skb,`
`156`	`156`	`nlh = nlmsg_put(ub->skb, 0, ub->qlen, 0,`
`157`	`157`	`size - NLMSG_ALIGN(sizeof(*nlh)), 0);`
`158`	`158`	`if (!nlh) {`
`159`		`- kfree(ub->skb);`
	`159`	`+ kfree_skb(ub->skb);`
`160`	`160`	`ub->skb = NULL;`
`161`	`161`	`goto unlock;`
`162`	`162`	`}`