diff options
author | Rich Felker <dalias@aerifal.cx> | 2014-06-17 13:56:54 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2014-06-17 13:56:54 -0400 |
commit | f4cc27601817f6decd773c24496bfbe892bcd244 (patch) | |
tree | 6e99f1143b72f742f52b59d34bed23200805082c /src/thread/x32 | |
parent | bfa09700b927c99cff5263483c1d7a4d1fe766e5 (diff) | |
download | musl-f4cc27601817f6decd773c24496bfbe892bcd244.tar.gz musl-f4cc27601817f6decd773c24496bfbe892bcd244.tar.xz musl-f4cc27601817f6decd773c24496bfbe892bcd244.zip |
multiple fixes to sh (superh) dynamic linker relocations
the following issues are fixed: - R_SH_REL32 was adding the load address of the module being relocated to the result. this seems to have been a mistake in the original port, since it does not match other dynamic linker implementations and since adding a difference between two addresses (the symbol value and the relocation address) to a load address does not make sense. - R_SH_TLS_DTPMOD32 was wrongly accepting an inline addend (i.e. using += rather than = on *reloc_addr) which makes no sense; addition is not an operation that's defined on module ids. - R_SH_TLS_DTPOFF32 and R_SH_TLS_TPOFF32 were wrongly using inline addends rather than the RELA-provided addends. in addition, handling of R_SH_GLOB_DAT, R_SH_JMP_SLOT, and R_SH_DIR32 are merged to all honor the addend. the first two should not need it for correct usage generated by toolchains, but other dynamic linkers allow addends here, and it simplifies the code anyway. these issues were spotted while reviewing the code for the purpose of refactoring this part of the dynamic linker. no testing was performed.
Diffstat (limited to 'src/thread/x32')
0 files changed, 0 insertions, 0 deletions