diff options
author | Owen Anderson <resistor@mac.com> | 2010-12-07 07:56:20 +0000 |
---|---|---|
committer | Owen Anderson <resistor@mac.com> | 2010-12-07 07:56:20 +0000 |
commit | 10e1b56e2c9b920d5f3509abb8a6df3815200839 (patch) | |
tree | 5531f3d84a8019dab7326efed2f2933e0e862255 /llvm/lib/Support/DynamicLibrary.cpp | |
parent | 182ea825a94aa5c8f3c13aea9cb532d86ab11cae (diff) | |
download | bcm5719-llvm-10e1b56e2c9b920d5f3509abb8a6df3815200839.tar.gz bcm5719-llvm-10e1b56e2c9b920d5f3509abb8a6df3815200839.zip |
Don't leak the mutex when loading dynamic libraries.
llvm-svn: 121119
Diffstat (limited to 'llvm/lib/Support/DynamicLibrary.cpp')
-rw-r--r-- | llvm/lib/Support/DynamicLibrary.cpp | 17 |
1 files changed, 5 insertions, 12 deletions
diff --git a/llvm/lib/Support/DynamicLibrary.cpp b/llvm/lib/Support/DynamicLibrary.cpp index cd9927a193a..455c3801cc6 100644 --- a/llvm/lib/Support/DynamicLibrary.cpp +++ b/llvm/lib/Support/DynamicLibrary.cpp @@ -61,17 +61,12 @@ using namespace llvm::sys; //=== independent code. //===----------------------------------------------------------------------===// -static SmartMutex<true>* HandlesMutex; static std::vector<void *> *OpenedHandles = 0; -static bool InitializeMutex() { - HandlesMutex = new SmartMutex<true>; - return HandlesMutex != 0; -} -static bool EnsureMutexInitialized() { - static bool result = InitializeMutex(); - return result; +static SmartMutex<true>& getMutex() { + static SmartMutex<true> HandlesMutex; + return HandlesMutex; } @@ -88,8 +83,7 @@ bool DynamicLibrary::LoadLibraryPermanently(const char *Filename, if (Filename == NULL) H = RTLD_DEFAULT; #endif - EnsureMutexInitialized(); - SmartScopedLock<true> Lock(*HandlesMutex); + SmartScopedLock<true> Lock(getMutex()); if (OpenedHandles == 0) OpenedHandles = new std::vector<void *>(); OpenedHandles->push_back(H); @@ -124,8 +118,7 @@ void* DynamicLibrary::SearchForAddressOfSymbol(const char* symbolName) { #if HAVE_DLFCN_H // Now search the libraries. - EnsureMutexInitialized(); - SmartScopedLock<true> Lock(*HandlesMutex); + SmartScopedLock<true> Lock(getMutex()); if (OpenedHandles) { for (std::vector<void *>::iterator I = OpenedHandles->begin(), E = OpenedHandles->end(); I != E; ++I) { |