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 2017/05/11 21:49:48 UTC
[trafficserver] 04/04: Coverity CID #1267799 Resource leak Coverity
CID #1241990 Resource leak
This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
commit 05423d2b859a2d3614e1b3c7719a0a69b0864e2c
Author: Phil Sorber <so...@apache.org>
AuthorDate: Thu May 11 14:38:15 2017 -0600
Coverity CID #1267799 Resource leak
Coverity CID #1241990 Resource leak
---
plugins/experimental/ssl_cert_loader/domain-tree.cc | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/plugins/experimental/ssl_cert_loader/domain-tree.cc b/plugins/experimental/ssl_cert_loader/domain-tree.cc
index 47324b3..b642ed7 100644
--- a/plugins/experimental/ssl_cert_loader/domain-tree.cc
+++ b/plugins/experimental/ssl_cert_loader/domain-tree.cc
@@ -173,10 +173,11 @@ DomainNameTree::insert(std::string key, void *payload, int order)
new_node->parent = node->parent;
new_node->children.push_back(node);
+ node->parent = new_node;
// Replace the node with new_node in the child list of the parent;
- for (std::deque<DomainNameNode *>::iterator iter = node->parent->children.begin(); iter != node->parent->children.end();
- ++iter) {
+ for (std::deque<DomainNameNode *>::iterator iter = new_node->parent->children.begin();
+ iter != new_node->parent->children.end(); ++iter) {
if (*(iter) == node) {
*(iter) = new_node;
}
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.