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;