[trafficserver] branch master updated: pop messages

[duke8253]

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

duke8253 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 40f94cc  pop messages
40f94cc is described below

commit 40f94cc062e39c15f21b97e75151b32253a08d97
Author: Fei Deng <f...@oath.com>
AuthorDate: Fri Mar 8 13:15:32 2019 -0600

    pop messages
---
 plugins/experimental/ssl_session_reuse/src/publish.cc | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/plugins/experimental/ssl_session_reuse/src/publish.cc 
b/plugins/experimental/ssl_session_reuse/src/publish.cc
index 8729204..c0235b3 100644
--- a/plugins/experimental/ssl_session_reuse/src/publish.cc
+++ b/plugins/experimental/ssl_session_reuse/src/publish.cc
@@ -239,6 +239,9 @@ RedisPublisher::runWorker()
     }
 
     Message &current_message(m_messageQueue.front());
+    if (!current_message.cleanup) {
+      m_messageQueue.pop_front();
+    }
 
     m_messageQueueMutex.unlock();
     ::sem_wait(&m_workerSem);
@@ -264,10 +267,6 @@ RedisPublisher::runWorker()
       }
     }
 
-    if (!current_message.cleanup) {
-      m_messageQueue.pop_front();
-    }
-
     clear_reply(current_reply);
     current_reply = nullptr;
   }