trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r..@apache.org
Subject [trafficserver] branch master updated: Fixes memory leak loading certs
Date Wed, 15 Apr 2020 16:18:11 GMT
This is an automated email from the ASF dual-hosted git repository.

rrm 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 9fb6f96  Fixes memory leak loading certs
9fb6f96 is described below

commit 9fb6f961baf31383b106d734f524e89406e2fa1e
Author: Randall Meyer <rrm@apache.org>
AuthorDate: Wed Apr 15 08:44:21 2020 -0700

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

diff --git a/iocore/net/SSLUtils.cc b/iocore/net/SSLUtils.cc
index cfcbd11..97ff0ff 100644
--- a/iocore/net/SSLUtils.cc
+++ b/iocore/net/SSLUtils.cc
@@ -2023,7 +2023,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;


Mime
View raw message