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

bcall 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 005a802  Ignore MgmtMessageHdr if ProcessManager is not running
005a802 is described below

commit 005a80292a487d516cd5fc8f63564e821358f585
Author: Masaori Koshiba <masa...@apache.org>
AuthorDate: Wed Oct 23 13:48:17 2019 +0900

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

diff --git a/mgmt/ProcessManager.cc b/mgmt/ProcessManager.cc
index 86fe41c..889a38f 100644
--- a/mgmt/ProcessManager.cc
+++ b/mgmt/ProcessManager.cc
@@ -240,6 +240,11 @@ ProcessManager::signalManager(int msg_id, const char 
*data_str)
 void
 ProcessManager::signalManager(int msg_id, const char *data_raw, int data_len)
 {
+  if (!this->running) {
+    Warning("MgmtMessageHdr is ignored. Because ProcessManager is not 
running");
+    return;
+  }
+
   MgmtMessageHdr *mh;
 
   mh           = (MgmtMessageHdr *)ats_malloc(sizeof(MgmtMessageHdr) + 
data_len);

Reply via email to