From d1247df266130561c9f1ab46a905ab04888ef6d9 Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Tue, 15 Mar 2016 11:38:56 +0000 Subject: [PATCH] Ensure that memory allocated for the ticket is freed If a call to EVP_DecryptUpdate fails then a memory leak could occur. Ensure that the memory is freed appropriately. Issue reported by Guido Vranken. Reviewed-by: Richard Levitte --- ssl/t1_lib.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ssl/t1_lib.c b/ssl/t1_lib.c index 8f5342b39f..fc7ebe41ca 100644 --- a/ssl/t1_lib.c +++ b/ssl/t1_lib.c @@ -3162,6 +3162,7 @@ static int tls_decrypt_ticket(SSL *s, const unsigned char *etick, if (sdec == NULL || EVP_DecryptUpdate(ctx, sdec, &slen, p, eticklen) <= 0) { EVP_CIPHER_CTX_free(ctx); + OPENSSL_free(sdec); return -1; } if (EVP_DecryptFinal(ctx, sdec + slen, &mlen) <= 0) {