diff options
| author | Dimitry Andric <dim@FreeBSD.org> | 2017-04-16 16:01:22 +0000 | 
|---|---|---|
| committer | Dimitry Andric <dim@FreeBSD.org> | 2017-04-16 16:01:22 +0000 | 
| commit | 71d5a2540a98c81f5bcaeb48805e0e2881f530ef (patch) | |
| tree | 5343938942df402b49ec7300a1c25a2d4ccd5821 /lib/Support/DynamicLibrary.cpp | |
| parent | 31bbf64f3a4974a2d6c8b3b27ad2f519caf74057 (diff) | |
Notes
Diffstat (limited to 'lib/Support/DynamicLibrary.cpp')
| -rw-r--r-- | lib/Support/DynamicLibrary.cpp | 21 | 
1 files changed, 14 insertions, 7 deletions
| diff --git a/lib/Support/DynamicLibrary.cpp b/lib/Support/DynamicLibrary.cpp index ced21e46afe8..92ce6185306a 100644 --- a/lib/Support/DynamicLibrary.cpp +++ b/lib/Support/DynamicLibrary.cpp @@ -9,8 +9,6 @@  //  //  This file implements the operating system DynamicLibrary concept.  // -// FIXME: This file leaks ExplicitSymbols and OpenedHandles! -//  //===----------------------------------------------------------------------===//  #include "llvm/Support/DynamicLibrary.h" @@ -51,7 +49,7 @@ using namespace llvm::sys;  //===          independent code.  //===----------------------------------------------------------------------===// -static DenseSet<void *> *OpenedHandles = nullptr; +static llvm::ManagedStatic<DenseSet<void *> > OpenedHandles;  DynamicLibrary DynamicLibrary::getPermanentLibrary(const char *filename,                                                     std::string *errMsg) { @@ -70,9 +68,6 @@ DynamicLibrary DynamicLibrary::getPermanentLibrary(const char *filename,      handle = RTLD_DEFAULT;  #endif -  if (!OpenedHandles) -    OpenedHandles = new DenseSet<void *>(); -    // If we've already loaded this library, dlclose() the handle in order to    // keep the internal refcount at +1.    if (!OpenedHandles->insert(handle).second) @@ -81,6 +76,18 @@ DynamicLibrary DynamicLibrary::getPermanentLibrary(const char *filename,    return DynamicLibrary(handle);  } +DynamicLibrary DynamicLibrary::addPermanentLibrary(void *handle, +                                                   std::string *errMsg) { +  SmartScopedLock<true> lock(*SymbolsMutex); +  // If we've already loaded this library, tell the caller. +  if (!OpenedHandles->insert(handle).second) { +    if (errMsg) *errMsg = "Library already loaded"; +    return DynamicLibrary(); +  } + +  return DynamicLibrary(handle); +} +  void *DynamicLibrary::getAddressOfSymbol(const char *symbolName) {    if (!isValid())      return nullptr; @@ -121,7 +128,7 @@ void* DynamicLibrary::SearchForAddressOfSymbol(const char *symbolName) {  #if defined(HAVE_DLFCN_H) && defined(HAVE_DLOPEN)    // Now search the libraries. -  if (OpenedHandles) { +  if (OpenedHandles.isConstructed()) {      for (DenseSet<void *>::iterator I = OpenedHandles->begin(),           E = OpenedHandles->end(); I != E; ++I) {        //lt_ptr ptr = lt_dlsym(*I, symbolName); | 
