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 2019/11/04 16:55:27 UTC

[trafficserver] branch 9.0.x updated: Ignore MgmtMessageHdr if ProcessManager is not running

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


The following commit(s) were added to refs/heads/9.0.x by this push:
     new 8573b23  Ignore MgmtMessageHdr if ProcessManager is not running
8573b23 is described below

commit 8573b234eaac86034f5e29532b44b31c4b555f1f
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Wed Oct 23 13:48:17 2019 +0900

    Ignore MgmtMessageHdr if ProcessManager is not running
    
    (cherry picked from commit 5936dcdda6d8f6695ce613e6022bf735732ca566)
---
 mgmt/ProcessManager.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/mgmt/ProcessManager.cc b/mgmt/ProcessManager.cc
index ef18e23..897cef1 100644
--- a/mgmt/ProcessManager.cc
+++ b/mgmt/ProcessManager.cc
@@ -266,6 +266,10 @@ ProcessManager::signalManager(int msg_id, std::string_view text)
 void
 ProcessManager::signalManager(MgmtMessageHdr *mh)
 {
+  if (!this->running) {
+    Warning("MgmtMessageHdr is ignored. Because ProcessManager is not running");
+    return;
+  }
   ink_release_assert(::enqueue(mgmt_signal_queue, mh));
 
 #if HAVE_EVENTFD