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/04/09 18:44:06 UTC

[trafficserver] branch 8.1.x updated: Fix memleak in ProcessManager::stop()

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

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


The following commit(s) were added to refs/heads/8.1.x by this push:
     new 0084ba1  Fix memleak in ProcessManager::stop()
0084ba1 is described below

commit 0084ba19f33df97b885fa8ebb553a82aa8ffa946
Author: Oknet Xu <xu...@skyguard.com.cn>
AuthorDate: Wed Mar 13 21:46:17 2019 +0800

    Fix memleak in ProcessManager::stop()
    
    (cherry picked from commit e27bcec1ba191668000b7f8d0cdaacc9c0e3c0da)
---
 mgmt/ProcessManager.cc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/mgmt/ProcessManager.cc b/mgmt/ProcessManager.cc
index 889a38f..d40d420 100644
--- a/mgmt/ProcessManager.cc
+++ b/mgmt/ProcessManager.cc
@@ -129,7 +129,9 @@ ProcessManager::stop()
     ats_free(sig);
   }
 
-  ats_free(mgmt_signal_queue);
+  LLQ *tmp_queue    = mgmt_signal_queue;
+  mgmt_signal_queue = nullptr;
+  delete_queue(tmp_queue);
 }
 
 /*