d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix a relocation bug in the ia64 ld.so. Weak function pointers in shared

Browse Source 
objects were not being correctly set to zero.  Instead, the function
descriptor pointer was set to the load address of the .so object.  This
caused gcc generated binaries to segfault on exit when crtbegin.asm's
_fini code tested the __cxa_finalize() function pointer for zero.

This is a bit of a hack because of a problem nearby workaround for
find_symdef and its quirks (failures) for local symbols.  This still
needs to be fixed.
This commit is contained in:
peter 2002-04-07 04:16:35 +00:00
parent 4a1195d2b7
commit 888c632b35
1 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
libexec/rtld-elf/ia64/reloc.c
View File

@ -136,7 +136,7 @@ reloc_non_plt_obj(Obj_Entry *obj_rtld, Obj_Entry *obj, const Elf_Rela *rela,
* to ensure this within a single object. If the
* caller's alloca failed, we don't even ensure that.
*/
const Elf_Sym *def;
const Elf_Sym *def, *ref;
const Obj_Entry *defobj;
struct fptr *fptr = 0;
Elf_Addr target, gp;
@ -153,8 +153,21 @@ reloc_non_plt_obj(Obj_Entry *obj_rtld, Obj_Entry *obj, const Elf_Rela *rela,
def = &obj->symtab[ELF_R_SYM(rela->r_info)];
defobj = obj;
}
target = (Elf_Addr) (defobj->relocbase + def->st_value);
gp = (Elf_Addr) defobj->pltgot;
/*
* If this is an undefined weak reference, we need to
* have a zero target,gp fptr, not pointing to relocbase.
* This isn't quite right. Maybe we should check
* explicitly for def == &sym_zero.
*/
if (def->st_value == 0 &&
(ref = obj->symtab + ELF_R_SYM(rela->r_info)) &&
ELF_ST_BIND(ref->st_info) == STB_WEAK) {
target = 0;
gp = 0;
} else {
target = (Elf_Addr) (defobj->relocbase + def->st_value);
gp = (Elf_Addr) defobj->pltgot;
}
/*
* Find the @fptr, using fptrs as a helper.