On Tue, 2013-11-26 at 15:23 +0100, Jan Kratochvil wrote:
Maybe you had a reason for it but we could also just replace
__libdwfl_module_getsym by dwfl_module_getsym_elf now. It would just require
changing
static inline GElf_Addr
dwfl_deadjust_st_value (Dwfl_Module *mod, struct dwfl_file *symfile,
GElf_Addr addr)
so that it accepts Elf * instead of struct dwfl_file * .
I admit I hadn't thought of that. Now I realize returning the Elf
without the bias is not as useful. For that the dwfl_file is also
useful. But we can still make the simplification you suggest. It does
cause a lot of small changes all over the place, hope that doesn't clash
too much with your pending patches. See attached. Also added one more
test to dwflsyms to make sure the returned bias is sane wrt the elf and
shndx.
Thanks,
Mark