commit 7bccbfcb7d3db5dbd57f4b7d10252e763dcaf9c5
author dinglei <libya_003@163.com> Wed Mar 25 10:59:55 2020 +0800
committer GitHub <noreply@github.com> Wed Mar 25 10:59:55 2020 +0800
tree a427ae81d2eda82d1c1bd01396cd8916b6f138d9
parent 81f36ecf3484b7eaf13782a1f833625c0071f127

[ISSUE #286] Add null pointer check before push consumer shutdown. (#287)

* Add null pointer check before push consumer shutdown.

src/consumer/DefaultMQPushConsumerImpl.cpp

diff --git a/src/consumer/DefaultMQPushConsumerImpl.cpp b/src/consumer/DefaultMQPushConsumerImpl.cpp
index 98ac401..40dc291 100644
--- a/src/consumer/DefaultMQPushConsumerImpl.cpp
+++ b/src/consumer/DefaultMQPushConsumerImpl.cpp
@@ -1112,8 +1112,13 @@
   m_DefaultMQProducerImpl->start(false);
 }
 void DefaultMQPushConsumerImpl::shutdownMessageTraceInnerProducer() {
-  LOG_INFO("Shutdown Message Trace Inner Producer In Consumer.");
-  m_DefaultMQProducerImpl->shutdown(false);
+  if (!getMessageTrace()) {
+    return;
+  }
+  if (m_DefaultMQProducerImpl) {
+    LOG_INFO("Shutdown Message Trace Inner Producer In Consumer.");
+    m_DefaultMQProducerImpl->shutdown(false);
+  }
 }
 bool DefaultMQPushConsumerImpl::hasConsumeMessageHook() {
   return !m_consumeMessageHookList.empty();