This is an automated email from the ASF dual-hosted git repository.

acanary pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new c21d754  SSLConfig mem leak fix (#7125)
c21d754 is described below

commit c21d754e404ac48b8602ea376e6981b7aaf81e60
Author: a-a-ron <acan...@verizonmedia.com>
AuthorDate: Tue Aug 25 09:20:26 2020 -0500

    SSLConfig mem leak fix (#7125)
    
    ASAN found this leak when doing a config reload.
---
 iocore/net/SSLConfig.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/iocore/net/SSLConfig.cc b/iocore/net/SSLConfig.cc
index 7f686ce..cfdcda9 100644
--- a/iocore/net/SSLConfig.cc
+++ b/iocore/net/SSLConfig.cc
@@ -367,6 +367,7 @@ SSLConfigParams::initialize()
             verify_server);
     verifyServerPolicy = YamlSNIConfig::Policy::DISABLED;
   }
+  ats_free(verify_server);
 
   REC_ReadConfigStringAlloc(verify_server, 
"proxy.config.ssl.client.verify.server.properties");
   if (strcmp(verify_server, "SIGNATURE") == 0) {

Reply via email to