summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBen Hutchings <ben@decadent.org.uk>2014-12-07 19:59:23 +0000
committerWilly Tarreau <w@1wt.eu>2014-12-13 15:16:18 +0100
commit7185a7199b3e6540e72c718db0ad1a91676e37d0 (patch)
tree383132eb0effc42edb6e9ddd6add9a55099aef0d
parent04b872de72b9ddc84d4a0feae477ca7a484af9c1 (diff)
sctp: Fix double-free introduced by bad backport in 2.6.32.62
One deletion was omitted from the backport of upstream commit c485658bae87 ("net: sctp: fix skb leakage in COOKIE ECHO path of chunk->auth_chunk"). Signed-off-by: Ben Hutchings <ben@decadent.org.uk> Signed-off-by: Willy Tarreau <w@1wt.eu>
-rw-r--r--net/sctp/sm_statefuns.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/net/sctp/sm_statefuns.c b/net/sctp/sm_statefuns.c
index d40ff4aabbf4..1d40672750ce 100644
--- a/net/sctp/sm_statefuns.c
+++ b/net/sctp/sm_statefuns.c
@@ -750,7 +750,6 @@ sctp_disposition_t sctp_sf_do_5_1D_ce(const struct sctp_endpoint *ep,
/* Make sure that we and the peer are AUTH capable */
if (!sctp_auth_enable || !new_asoc->peer.auth_capable) {
- kfree_skb(chunk->auth_chunk);
sctp_association_free(new_asoc);
return sctp_sf_pdiscard(ep, asoc, type, arg, commands);
}