generic-library/bionic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

am 38ff8822: Merge "Revert "Use local copy of the library name on dlopen""

Browse Source 
* commit '38ff8822763a30d4c9033a38ec8b71fcd17aba87':
  Revert "Use local copy of the library name on dlopen"
This commit is contained in:
Dmitriy Ivanov 2014-11-01 02:35:51 +00:00 committed by Android Git Automerger
commit 75cd66aaf6
1 changed files with 2 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
linker/linker.cpp
View File

@ -107,8 +107,6 @@ static const char* const kDefaultLdPaths[] = {
#define LDPRELOAD_BUFSIZE (LDPRELOAD_MAX*64)
#define LDPRELOAD_MAX 8
#define MAX_PATH_LEN 512
static char g_ld_library_paths_buffer[LDPATH_BUFSIZE];
static const char* g_ld_library_paths[LDPATH_MAX + 1];
@ -744,7 +742,7 @@ ElfW(Sym)* dladdr_find_symbol(soinfo* si, const void* addr) {
}
static int open_library_on_path(const char* name, const char* const paths[]) {
char buf[MAX_PATH_LEN];
char buf[512];
for (size_t i = 0; paths[i] != nullptr; ++i) {
int n = __libc_format_buffer(buf, sizeof(buf), "%s/%s", paths[i], name);
if (n < 0 || n >= static_cast<int>(sizeof(buf))) {
@ -1123,18 +1121,8 @@ soinfo* do_dlopen(const char* name, int flags, const android_dlextinfo* extinfo)
return nullptr;
}
}
size_t name_len = strlen(name);
if (name_len >= MAX_PATH_LEN) {
DL_ERR("library name \"%s\" is too long", name);
return nullptr;
}
char local_name[name_len+1];
strlcpy(local_name, name, name_len+1);
protect_data(PROT_READ | PROT_WRITE);
soinfo* si = find_library(local_name, flags, extinfo);
soinfo* si = find_library(name, flags, extinfo);
if (si != nullptr) {
si->CallConstructors();
}