[Lldb-commits] [lldb] r250493 - Add a missing Mutex Locker in ResetCurrentInlinedDepth

[Keno Fischer via lldb-commits]

Author: kfischer
Date: Fri Oct 16 00:17:25 2015
New Revision: 250493

URL: http://llvm.org/viewvc/llvm-project?rev=250493&view=rev
Log:
Add a missing Mutex Locker in ResetCurrentInlinedDepth

Summary: This function reads m_frames, so it needs to lock the
mutex like all other function that access m_frames.
Found using AddressSanitizer.

Reviewers: jingham, clayborg

Subscribers: lldb-commits

Differential Revision: http://reviews.llvm.org/D13638

Modified:
    lldb/trunk/source/Target/StackFrameList.cpp

Modified: lldb/trunk/source/Target/StackFrameList.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Target/StackFrameList.cpp?rev=250493&r1=250492&r2=250493&view=diff
==============================================================================
--- lldb/trunk/source/Target/StackFrameList.cpp (original)
+++ lldb/trunk/source/Target/StackFrameList.cpp Fri Oct 16 00:17:25 2015
@@ -105,6 +105,8 @@ StackFrameList::GetCurrentInlinedDepth (
 void
 StackFrameList::ResetCurrentInlinedDepth ()
 {
+    Mutex::Locker locker (m_mutex);
+
     if (m_show_inlined_frames)
     {        
         GetFramesUpTo(0);