tequilaOS/platform_bionic
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

dlfcn.h: dlclose doc ~'do not use'

Browse source 
There are a lot of bugs about this over the years, too many to
reference here. Though, I referenced b/176065420 to understand
exactly why it's problematic and what the future direction may
be.

Fixes: 307859642
Test: N/A
Change-Id: Ida31fe622309a7f9b2cd55e5bbb3569fc5aded0e
This commit is contained in:
Steven Moreland 2023-11-01 00:16:31 +00:00 committed by Elliott Hughes
parent f971dc6b4a
commit f61b2b16b8
1 changed files with 23 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
libc/include/dlfcn.h
View file

@ -46,8 +46,31 @@ typedef struct {
} Dl_info;
void* _Nullable dlopen(const char* _Nullable __filename, int __flag);
/**
* [dlclose(3)](http://man7.org/linux/man-pages/man3/dlclose.3.html)
* decrements the reference count for the given shared library (and
* any libraries brought in by that library's DT_NEEDED entries).
*
* If a library's reference count hits zero, it may be unloaded.
* Code that relies on this is not portable, and may not work on
* future versions of Android.
*
* dlclose() is dangerous because function pointers may or may not
* be rendered invalid, global data may or may not be rendered invalid,
* and memory may or may not leak. Code with global constructors is
* especially problematic. Instead of dlclose, prefer to leave the
* library open or, if cleanup is necessary, dlopen() the library in
* a child process which can later be killed by the parent or call
* exit() itself.
*
* Returns 0 on success, and returns -1 on failure, in which case
* dlerror() can be used to retrieve the specific error.
*/
int dlclose(void* _Nonnull __handle);
char* _Nullable dlerror(void);
/* (RTLD_DEFAULT is null for LP64, but -1 for LP32) */
void* _Nullable dlsym(void* __BIONIC_COMPLICATED_NULLNESS __handle, const char* _Nullable __symbol);
/* (RTLD_DEFAULT is null for LP64, but -1 for LP32) */