From: Kurt Roeckx <kurt@roeckx.be>
Date: Sun, 18 Dec 2016 11:48:49 +0000 (+0100)
Subject: Fix memory leak in tls_parse_stoc_key_share
X-Git-Tag: OpenSSL_1_1_1-pre1~2832
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a1d6a0b6e6cc410ee709b4cefd56708e977af3be;p=openssl

Fix memory leak in tls_parse_stoc_key_share

Found by oss-fuzz

Reviewed-by: Richard Levitte <levitte@openssl.org>
GH: #2102
---

diff --git a/ssl/statem/extensions_clnt.c b/ssl/statem/extensions_clnt.c
index 70d90e7b4f..6d2ed23c89 100644
--- a/ssl/statem/extensions_clnt.c
+++ b/ssl/statem/extensions_clnt.c
@@ -1027,6 +1027,7 @@ int tls_parse_stoc_key_share(SSL *s, PACKET *pkt, int *al)
                                         PACKET_remaining(&encoded_pt))) {
         *al = SSL_AD_DECODE_ERROR;
         SSLerr(SSL_F_TLS_PARSE_STOC_KEY_SHARE, SSL_R_BAD_ECPOINT);
+        EVP_PKEY_free(skey);
         return 0;
     }