From 1a2933c71a0fbe8c729ffe5567ad02124d119005 Mon Sep 17 00:00:00 2001 From: "henrike@webrtc.org" Date: Thu, 6 Oct 2011 17:55:56 +0000 Subject: [PATCH] Fixes a Valgrind warning triggering when the number of pending messages hit the limit. Review URL: http://webrtc-codereview.appspot.com/200002 git-svn-id: http://webrtc.googlecode.com/svn/trunk@705 4adac7df-926f-26a2-2b94-8c16560cd09d --- src/system_wrappers/source/trace_impl.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/system_wrappers/source/trace_impl.cc b/src/system_wrappers/source/trace_impl.cc index c9be30ab9f..2bbd6f815f 100644 --- a/src/system_wrappers/source/trace_impl.cc +++ b/src/system_wrappers/source/trace_impl.cc @@ -607,9 +607,12 @@ void TraceImpl::AddMessageToList( if(_nextFreeIdx[_activeQueue] == WEBRTC_TRACE_MAX_QUEUE-1) { - // Loggin more messages than can be worked off. Log a warning. + // Logging more messages than can be worked off. Log a warning. + const char warning_msg[] = "WARNING MISSING TRACE MESSAGES\n"; + _level[_activeQueue][_nextFreeIdx[_activeQueue]] = kTraceWarning; + _length[_activeQueue][_nextFreeIdx[_activeQueue]] = strlen(warning_msg); memcpy(_messageQueue[_activeQueue][_nextFreeIdx[_activeQueue]], - "WARNING MISSING TRACE MESSAGES\n", 32); + warning_msg, _length[_activeQueue][idx]); _nextFreeIdx[_activeQueue]++; } }