diff options
| author | Jan Beulich <jbeulich@suse.com> | 2023-08-25 14:56:44 +0200 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2023-08-25 14:56:44 +0200 |
| commit | 67694446f772ae95d330419eb6d01e38e6d24612 (patch) | |
| tree | 21119de766ad7554bd06371b5713ab4f58917c96 /bfd/coff-sh.c | |
| parent | 183440b0aa1cd752c324a2f4c16a620bd938d9ad (diff) | |
| download | gdb-67694446f772ae95d330419eb6d01e38e6d24612.zip gdb-67694446f772ae95d330419eb6d01e38e6d24612.tar.gz gdb-67694446f772ae95d330419eb6d01e38e6d24612.tar.bz2 | |
bfd: correct relocation handling for objcopy COFF -> ELF
While documented to not be reliable, it is still odd for objcopy to
silently produce bad output when converting COFF/PE object files to ELF
ones. The issue there is that relocation addends all are screwed up by
subtracting the symbol's section offset. In the COFF/PE world, to my
knowledge, section contents stores the addends alone, not the result of
symbol value plus addend. Hence the compensation talked about in a
comment ahead of the sole use site of CALC_ADDEND() may need to account
for the VMA (which is always zero for object files anyway), but not for
the symbol value.
The coff-sh.c adjustment is based upon guessing that behavior there is
the same. Note also how coff-aarch64.c short-circuits CALC_ADDEND()
altogether, which may suggest that a much simpler macro might do for the
COFF_WITH_PE case in the three arch-specific files touched here.
For (at least) Arm/WinCE this actually results in more appropriate
objdump output as well, as can be seen in the one testcase which has its
expectations adjusted (the generated binary doesn't change).
Diffstat (limited to 'bfd/coff-sh.c')
| -rw-r--r-- | bfd/coff-sh.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/bfd/coff-sh.c b/bfd/coff-sh.c index 14f502e..a20c011 100644 --- a/bfd/coff-sh.c +++ b/bfd/coff-sh.c @@ -544,7 +544,8 @@ sh_coff_reloc_name_lookup (bfd *abfd ATTRIBUTE_UNUSED, cache_ptr->addend = 0; \ else if (ptr && bfd_asymbol_bfd (ptr) == abfd \ && ptr->section != (asection *) NULL) \ - cache_ptr->addend = - (ptr->section->vma + ptr->value); \ + cache_ptr->addend = - (ptr->section->vma \ + + COFF_PE_ADDEND_BIAS (ptr)); \ else \ cache_ptr->addend = 0; \ if ((reloc).r_type == R_SH_SWITCH8 \ |