From 0ea112f104215ccba8d89c839cdeded6e3d49e59 Mon Sep 17 00:00:00 2001 From: Owen Anderson Date: Tue, 7 Dec 2010 07:56:20 +0000 Subject: [PATCH] Don't leak the mutex when loading dynamic libraries. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@121119 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Support/DynamicLibrary.cpp | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/lib/Support/DynamicLibrary.cpp b/lib/Support/DynamicLibrary.cpp index cd9927a193a..455c3801cc6 100644 --- a/lib/Support/DynamicLibrary.cpp +++ b/lib/Support/DynamicLibrary.cpp @@ -61,17 +61,12 @@ using namespace llvm::sys; //=== independent code. //===----------------------------------------------------------------------===// -static SmartMutex* HandlesMutex; static std::vector *OpenedHandles = 0; -static bool InitializeMutex() { - HandlesMutex = new SmartMutex; - return HandlesMutex != 0; -} -static bool EnsureMutexInitialized() { - static bool result = InitializeMutex(); - return result; +static SmartMutex& getMutex() { + static SmartMutex HandlesMutex; + return HandlesMutex; } @@ -88,8 +83,7 @@ bool DynamicLibrary::LoadLibraryPermanently(const char *Filename, if (Filename == NULL) H = RTLD_DEFAULT; #endif - EnsureMutexInitialized(); - SmartScopedLock Lock(*HandlesMutex); + SmartScopedLock Lock(getMutex()); if (OpenedHandles == 0) OpenedHandles = new std::vector(); OpenedHandles->push_back(H); @@ -124,8 +118,7 @@ void* DynamicLibrary::SearchForAddressOfSymbol(const char* symbolName) { #if HAVE_DLFCN_H // Now search the libraries. - EnsureMutexInitialized(); - SmartScopedLock Lock(*HandlesMutex); + SmartScopedLock Lock(getMutex()); if (OpenedHandles) { for (std::vector::iterator I = OpenedHandles->begin(), E = OpenedHandles->end(); I != E; ++I) { -- 2.34.1