commit 4ba6c26623a2ba1c59adf2171308b92d04f6f7a1
author Niels Möller <nisse@webrtc.org> Wed Oct 24 13:13:07 2018
committer Commit Bot <commit-bot@chromium.org> Thu Oct 25 13:36:32 2018
tree f5df1818f542aae430d2cba83add1c375fda5b8c
parent 9516c385386f01b6c61bfbcc03801e601a6016e7

Delete MessageData when a message is posted to a quitting MessageQueue

Bug: webrtc:9913
Change-Id: Id60f537eb0049995c9f0837e3a03ca3a3dd90577
Reviewed-on: https://webrtc-review.googlesource.com/c/107639
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25365}

rtc_base/messagequeue.cc

diff --git a/rtc_base/messagequeue.cc b/rtc_base/messagequeue.cc
index d710ba5..204952a 100644
--- a/rtc_base/messagequeue.cc
+++ b/rtc_base/messagequeue.cc
@@ -350,8 +350,10 @@
                         uint32_t id,
                         MessageData* pdata,
                         bool time_sensitive) {
-  if (IsQuitting())
+  if (IsQuitting()) {
+    delete pdata;
     return;
+  }
 
   // Keep thread safe
   // Add the message to the end of the queue
@@ -407,6 +409,7 @@
                                uint32_t id,
                                MessageData* pdata) {
   if (IsQuitting()) {
+    delete pdata;
     return;
   }