diff options
| author | Eric Dumazet <edumazet@google.com> | 2013-06-24 06:26:00 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-06-25 16:07:44 -0700 |
| commit | bd8a7036c06cf15779b31a5397d4afcb12be81ea (patch) | |
| tree | d78794e12fb4ac0005f019e6a51e797230d38970 /net/ipv4 | |
| parent | 2b7a5db060f5e7d7f79bc45f818e08b8f244a3b8 (diff) | |
gre: fix a possible skb leak
commit 68c331631143 ("v4 GRE: Add TCP segmentation offload for GRE")
added a possible skb leak, because it frees only the head of segment
list, in case a skb_linearize() call fails.
This patch adds a kfree_skb_list() helper to fix the bug.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Pravin B Shelar <pshelar@nicira.com>
Cc: Daniel Borkmann <dborkman@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
| -rw-r--r-- | net/ipv4/gre.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/gre.c b/net/ipv4/gre.c index b2e805af9b8..7856d1651d0 100644 --- a/net/ipv4/gre.c +++ b/net/ipv4/gre.c @@ -178,7 +178,7 @@ static struct sk_buff *gre_gso_segment(struct sk_buff *skb, err = __skb_linearize(skb); if (err) { - kfree_skb(segs); + kfree_skb_list(segs); segs = ERR_PTR(err); goto out; } |