From fbdf0299dc98bc611d854c0a62c6ab1810d856fc Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Thu, 5 May 2016 20:59:03 +0100 Subject: [PATCH] Free any existing SRTP connection profile When setting a new SRTP connection profile using SSL_CTX_set_tlsext_use_srtp() or SSL_set_tlsext_use_srtp() we should free any existing profile first to avoid a memory leak. Reviewed-by: Kurt Roeckx --- ssl/d1_srtp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ssl/d1_srtp.c b/ssl/d1_srtp.c index f969fb10b1..e49aea9be1 100644 --- a/ssl/d1_srtp.c +++ b/ssl/d1_srtp.c @@ -197,6 +197,8 @@ static int ssl_ctx_make_profiles(const char *profiles_string, ptr = col + 1; } while (col); + sk_SRTP_PROTECTION_PROFILE_free(*out); + *out = profiles; return 0; -- 2.25.1