diff options
Diffstat (limited to 'lldb/source/Breakpoint/Breakpoint.cpp')
| -rw-r--r-- | lldb/source/Breakpoint/Breakpoint.cpp | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/lldb/source/Breakpoint/Breakpoint.cpp b/lldb/source/Breakpoint/Breakpoint.cpp index 5d0cbd47e20..789118b3a40 100644 --- a/lldb/source/Breakpoint/Breakpoint.cpp +++ b/lldb/source/Breakpoint/Breakpoint.cpp @@ -317,6 +317,7 @@ Breakpoint::ClearAllBreakpointSites () void Breakpoint::ModulesChanged (ModuleList &module_list, bool load, bool delete_locations) { + Mutex::Locker modules_mutex(module_list.GetMutex()); if (load) { // The logic for handling new modules is: @@ -332,11 +333,11 @@ Breakpoint::ModulesChanged (ModuleList &module_list, bool load, bool delete_loca // resolving breakpoints will add new locations potentially. const size_t num_locs = m_locations.GetSize(); - - for (size_t i = 0; i < module_list.GetSize(); i++) + size_t num_modules = module_list.GetSize(); + for (size_t i = 0; i < num_modules; i++) { bool seen = false; - ModuleSP module_sp (module_list.GetModuleAtIndex (i)); + ModuleSP module_sp (module_list.GetModuleAtIndexUnlocked (i)); if (!m_filter_sp->ModulePasses (module_sp)) continue; @@ -403,10 +404,11 @@ Breakpoint::ModulesChanged (ModuleList &module_list, bool load, bool delete_loca shared_from_this()); else removed_locations_event = NULL; - - for (size_t i = 0; i < module_list.GetSize(); i++) + + size_t num_modules = module_list.GetSize(); + for (size_t i = 0; i < num_modules; i++) { - ModuleSP module_sp (module_list.GetModuleAtIndex (i)); + ModuleSP module_sp (module_list.GetModuleAtIndexUnlocked (i)); if (m_filter_sp->ModulePasses (module_sp)) { size_t loc_idx = 0; |
