Fixed an issue that could happen during global object destruction in our
map that tracks all live Module classes. We must leak our mutex for our
collection class as it might be destroyed in an order we can't control.
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@149131 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/include/lldb/Core/Module.h b/include/lldb/Core/Module.h
index aabdd4a..bf8ed40 100644
--- a/include/lldb/Core/Module.h
+++ b/include/lldb/Core/Module.h
@@ -64,7 +64,7 @@
static Module *
GetAllocatedModuleAtIndex (size_t idx);
- static Mutex &
+ static Mutex *
GetAllocationModuleCollectionMutex();
//------------------------------------------------------------------
diff --git a/source/Commands/CommandObjectTarget.cpp b/source/Commands/CommandObjectTarget.cpp
index 29f5b9f..8b0fd17 100644
--- a/source/Commands/CommandObjectTarget.cpp
+++ b/source/Commands/CommandObjectTarget.cpp
@@ -1632,7 +1632,7 @@
if (check_global_list && num_matches == 0)
{
// Check the global list
- Mutex::Locker locker(Module::GetAllocationModuleCollectionMutex().GetMutex());
+ Mutex::Locker locker(Module::GetAllocationModuleCollectionMutex());
const uint32_t num_modules = Module::GetNumberAllocatedModules();
ModuleSP module_sp;
for (uint32_t image_idx = 0; image_idx<num_modules; ++image_idx)
@@ -2051,7 +2051,7 @@
else
{
// Check the global list
- Mutex::Locker locker(Module::GetAllocationModuleCollectionMutex().GetMutex());
+ Mutex::Locker locker(Module::GetAllocationModuleCollectionMutex());
result.AppendWarningWithFormat("Unable to find an image that matches '%s'.\n", arg_cstr);
}
@@ -2785,7 +2785,7 @@
if (use_global_module_list)
{
- locker.Reset (Module::GetAllocationModuleCollectionMutex().GetMutex());
+ locker.Reset (Module::GetAllocationModuleCollectionMutex()->GetMutex());
num_modules = Module::GetNumberAllocatedModules();
}
else
diff --git a/source/Core/Module.cpp b/source/Core/Module.cpp
index ea16376..62b25c5 100644
--- a/source/Core/Module.cpp
+++ b/source/Core/Module.cpp
@@ -43,11 +43,18 @@
return *g_module_collection;
}
-Mutex &
+Mutex *
Module::GetAllocationModuleCollectionMutex()
{
- static Mutex g_module_collection_mutex(Mutex::eMutexTypeRecursive);
- return g_module_collection_mutex;
+ // NOTE: The mutex below must be leaked since the global module list in
+ // the ModuleList class will get torn at some point, and we can't know
+ // if it will tear itself down before the "g_module_collection_mutex" below
+ // will. So we leak a Mutex object below to safeguard against that
+
+ static Mutex *g_module_collection_mutex = NULL;
+ if (g_module_collection_mutex == NULL)
+ g_module_collection_mutex = new Mutex (Mutex::eMutexTypeRecursive); // NOTE: known leak
+ return g_module_collection_mutex;
}
size_t