diff options
| author | Ben Hutchings <ben@decadent.org.uk> | 2014-12-07 19:59:23 +0000 |
|---|---|---|
| committer | Willy Tarreau <w@1wt.eu> | 2014-12-13 15:16:18 +0100 |
| commit | 7185a7199b3e6540e72c718db0ad1a91676e37d0 (patch) | |
| tree | 383132eb0effc42edb6e9ddd6add9a55099aef0d | |
| parent | 04b872de72b9ddc84d4a0feae477ca7a484af9c1 (diff) | |
| download | lwn-7185a7199b3e6540e72c718db0ad1a91676e37d0.tar.gz lwn-7185a7199b3e6540e72c718db0ad1a91676e37d0.zip | |
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.c | 1 |
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); } |