From: Patrick McHardy <kaber@trash.net>
Date: Mon, 13 Mar 2006 04:33:22 +0000 (-0800)
Subject: [NET_SCHED]: act_api: fix skb leak in error path
X-Git-Tag: tegra-9.12.3~22861
X-Git-Url: http://nv-tegra.nvidia.com/gitweb/?p=linux-2.6.git;a=commitdiff_plain;h=f6e57464df7ba0e91a23b0854adb56852dcd58a7;hp=406dbfc9ae0e6b7eb4da4a52c0a6556a0d7e6a2f

[NET_SCHED]: act_api: fix skb leak in error path

The skb is allocated by the function, so it needs to be freed instead
of trimmed on overrun.

Coverity #614

Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
---

diff --git a/net/sched/act_api.c b/net/sched/act_api.c
index 792ce59..2ffa11c 100644
--- a/net/sched/act_api.c
+++ b/net/sched/act_api.c
@@ -707,7 +707,7 @@ static int tcf_add_notify(struct tc_action *a, u32 pid, u32 seq, int event,
 
 rtattr_failure:
 nlmsg_failure:
-	skb_trim(skb, b - skb->data);
+	kfree_skb(skb);
 	return -1;
 }