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>.