From 6eb53f71d6930f0f92c69d128cb01d108c9de1d6 Mon Sep 17 00:00:00 2001 From: "hclam@chromium.org" Date: Thu, 20 Jun 2013 23:01:39 +0000 Subject: [PATCH] Fix memory bot failure Exit the method with critical setting held. This should make the memory bot happy. TBR=pwestin@webrtc.org Review URL: https://webrtc-codereview.appspot.com/1704005 git-svn-id: http://webrtc.googlecode.com/svn/trunk@4251 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/modules/pacing/paced_sender.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/webrtc/modules/pacing/paced_sender.cc b/webrtc/modules/pacing/paced_sender.cc index 2fdc5cfa4..4ffe86a66 100644 --- a/webrtc/modules/pacing/paced_sender.cc +++ b/webrtc/modules/pacing/paced_sender.cc @@ -260,13 +260,12 @@ int32_t PacedSender::Process() { const bool success = callback_->TimeToSendPacket(ssrc, sequence_number, capture_time_ms); + critsect_->Enter(); // If packet cannt be sent then keep it in packet list and exit early. // There's no need to send more packets. if (!success) { return 0; } - - critsect_->Enter(); packet_list->pop_front(); const bool last_packet = packet_list->empty() || packet_list->front().capture_time_ms_ > capture_time_ms;