From d709a4787df1319567bfc64993225f6280ddcf0a Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Thu, 16 Nov 2006 20:52:28 +0000 Subject: revert rev 16527 changes to ldso/ldso/dl-hash.c --- ldso/ldso/dl-hash.c | 62 +++++++++++++++++------------------------------------ 1 file changed, 20 insertions(+), 42 deletions(-) (limited to 'ldso') diff --git a/ldso/ldso/dl-hash.c b/ldso/ldso/dl-hash.c index 26022ff79..4fd7ba0b7 100644 --- a/ldso/ldso/dl-hash.c +++ b/ldso/ldso/dl-hash.c @@ -123,6 +123,7 @@ struct elf_resolve *_dl_add_elf_hash_table(const char *libname, return tpnt; } + /* * This function resolves externals, and this is either called when we process * relocations or when we call an entry in the PLT table for the first time. @@ -166,53 +167,30 @@ char *_dl_find_hash(const char *name, struct dyn_elf *rpnt, struct elf_resolve * strtab = (char *) (tpnt->dynamic_info[DT_STRTAB]); for (si = tpnt->elf_buckets[hn]; si != STN_UNDEF; si = tpnt->chains[si]) { - char *result; sym = &symtab[si]; - if (sym->st_shndx == SHN_UNDEF) - continue; - if (ELF_ST_TYPE(sym->st_info) > STT_FUNC -#if defined(__arm__) || defined(__thumb__) - /* On ARM (only) STT_ARM_TFUNC is a function - * and has a value >STT_FUNC, so this must - * be checked specially. - */ - && ELF_ST_TYPE(sym->st_info) != STT_ARM_TFUNC -#endif - ) - continue; - if (_dl_strcmp(strtab + sym->st_name, name) != 0) - continue; + if (type_class & (sym->st_shndx == SHN_UNDEF)) + continue; + if (_dl_strcmp(strtab + sym->st_name, name) != 0) + continue; + if (sym->st_value == 0) + continue; + if (ELF_ST_TYPE(sym->st_info) > STT_FUNC) + continue; + + switch (ELF_ST_BIND(sym->st_info)) { + case STB_WEAK: #if 0 - /* I don't know how to write this test - need to test shndx - * to see if it is the PLT for this module. - */ - if ((type_class & ELF_RTYPE_CLASS_PLT) && some test) - continue; -#endif - -#if defined(__arm__) || defined(__thumb__) - /* On ARM the caller needs to know that STT_ARM_TFUNC - * is a thumb function call, this is now indicated by - * setting the low bit of the value (and newer binutils - * will do this and record STT_FUNC). - */ - result = (char*)tpnt->loadaddr + (sym->st_value | - (ELF_ST_TYPE(sym->st_info) == STT_ARM_TFUNC)); -#else - result = (char*)tpnt->loadaddr + sym->st_value; +/* Perhaps we should support old style weak symbol handling + * per what glibc does when you export LD_DYNAMIC_WEAK */ + if (!weak_result) + weak_result = (char *) DL_RELOC_ADDR(tpnt->loadaddr, sym->st_value); + break; #endif - switch (ELF_ST_BIND(sym->st_info)) { - case STB_WEAK: - /* Record for use later if we can't find a global. */ - if (!weak_result) - weak_result = result; - break; - - case STB_GLOBAL: - return result; + case STB_GLOBAL: + return (char*) DL_RELOC_ADDR(tpnt->loadaddr, sym->st_value); default: /* Local symbols not handled here */ - break; + break; } } } -- cgit v1.2.3