summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric W. Biederman <ebiederm@xmission.com>2015-03-02 06:59:57 +0100
committerDavid S. Miller <davem@davemloft.net>2015-03-02 22:43:39 +0100
commite18dbd059319ac8d3fc0a0a23f6a8abcb9a59a31 (patch)
tree270c4f2a683cc173ce003813483125046ec880b0
parentebpf: move CONFIG_BPF_SYSCALL-only function declarations (diff)
downloadlinux-e18dbd059319ac8d3fc0a0a23f6a8abcb9a59a31.tar.xz
linux-e18dbd059319ac8d3fc0a0a23f6a8abcb9a59a31.zip
ax25: In ax25_rebuild_header add missing kfree_skb
In the unlikely (impossible?) event that we attempt to transmit an ax25 packet over a non-ax25 device free the skb so we don't leak it. Cc: Ralf Baechle <ralf@linux-mips.org> Cc: linux-hams@vger.kernel.org Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/ax25/ax25_ip.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/net/ax25/ax25_ip.c b/net/ax25/ax25_ip.c
index 67de6b33f2c3..db3c283821d1 100644
--- a/net/ax25/ax25_ip.c
+++ b/net/ax25/ax25_ip.c
@@ -129,6 +129,7 @@ int ax25_rebuild_header(struct sk_buff *skb)
dev = skb->dev;
if ((ax25_dev = ax25_dev_ax25dev(dev)) == NULL) {
+ kfree_skb(skb);
goto put;
}