From ff244c1d9754b0632401a2bd36612afb7392cf44 Mon Sep 17 00:00:00 2001
From: Jim Ingham <jingham@apple.com>
Date: Wed, 29 Jul 2015 00:40:36 +0000
Subject: [PATCH] Lock the owners mutex in the BreakpointSite before updating
 the hit counts.  If you delete a breakpoint belonging to a site just as you
 are processing a hit on that site, you could cause the BreakpointSite loop to
 access a now deleted location.

<rdar://problem/19310323>

llvm-svn: 243507
---
 lldb/source/Breakpoint/BreakpointSite.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lldb/source/Breakpoint/BreakpointSite.cpp b/lldb/source/Breakpoint/BreakpointSite.cpp
index e9ce812..95b67b0 100644
--- a/lldb/source/Breakpoint/BreakpointSite.cpp
+++ b/lldb/source/Breakpoint/BreakpointSite.cpp
@@ -205,6 +205,7 @@ BreakpointSite::ValidForThisThread (Thread *thread)
 void
 BreakpointSite::BumpHitCounts()
 {
+    Mutex::Locker locker(m_owners_mutex);
     for (BreakpointLocationSP loc_sp : m_owners.BreakpointLocations())
     {
         loc_sp->BumpHitCount();
-- 
2.7.4