You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2020/05/20 16:45:37 UTC

[trafficserver] 02/06: Fixes memory leak loading certs

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

zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit 7c08b051cd8a70a0e271c29b8f4d55dffa78529e
Author: Randall Meyer <rr...@apache.org>
AuthorDate: Wed Apr 15 08:44:21 2020 -0700

    Fixes memory leak loading certs
    
    This leak was introduced by f729c9dc41ff1635132f4bdc6331ce826f3bc2fe
    
    (cherry picked from commit 9fb6f961baf31383b106d734f524e89406e2fa1e)
---
 iocore/net/SSLUtils.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/iocore/net/SSLUtils.cc b/iocore/net/SSLUtils.cc
index 2c32232..6b3774d 100644
--- a/iocore/net/SSLUtils.cc
+++ b/iocore/net/SSLUtils.cc
@@ -2012,7 +2012,9 @@ SSLMultiCertConfigLoader::load_certs_and_cross_reference_names(std::vector<X509
           name_set.insert(dns.get());
         }
       }
+      sk_GENERAL_NAME_pop_free(names, GENERAL_NAME_free);
     }
+
     if (first_pass) {
       first_pass   = false;
       common_names = name_set;