33 lines
1.2 KiB
Diff
33 lines
1.2 KiB
Diff
From: "David S. Miller" <davem@davemloft.net>
|
|
Date: Sun, 4 Jun 2017 21:41:10 -0400
|
|
Subject: ipv6: Fix leak in ipv6_gso_segment().
|
|
Origin: https://git.kernel.org/linus/e3e86b5119f81e5e2499bea7ea1ebe8ac6aab789
|
|
Bug-Debian-Security: https://security-tracker.debian.org/tracker/CVE-2017-9074
|
|
|
|
If ip6_find_1stfragopt() fails and we return an error we have to free
|
|
up 'segs' because nobody else is going to.
|
|
|
|
Fixes: 2423496af35d ("ipv6: Prevent overrun when parsing v6 header options")
|
|
Reported-by: Ben Hutchings <ben@decadent.org.uk>
|
|
Signed-off-by: David S. Miller <davem@davemloft.net>
|
|
---
|
|
net/ipv6/ip6_offload.c | 4 +++-
|
|
1 file changed, 3 insertions(+), 1 deletion(-)
|
|
|
|
diff --git a/net/ipv6/ip6_offload.c b/net/ipv6/ip6_offload.c
|
|
index 280268f1dd7b..cdb3728faca7 100644
|
|
--- a/net/ipv6/ip6_offload.c
|
|
+++ b/net/ipv6/ip6_offload.c
|
|
@@ -116,8 +116,10 @@ static struct sk_buff *ipv6_gso_segment(struct sk_buff *skb,
|
|
|
|
if (udpfrag) {
|
|
int err = ip6_find_1stfragopt(skb, &prevhdr);
|
|
- if (err < 0)
|
|
+ if (err < 0) {
|
|
+ kfree_skb_list(segs);
|
|
return ERR_PTR(err);
|
|
+ }
|
|
fptr = (struct frag_hdr *)((u8 *)ipv6h + err);
|
|
fptr->frag_off = htons(offset);
|
|
if (skb->next)
|