diff options
| author | Alan Modra <amodra@gmail.com> | 2015-08-20 12:02:45 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2015-08-20 12:02:45 +0930 |
| commit | 9215b98bb27c071386a277f5578dbb17569a1471 (patch) | |
| tree | 62014c06b7d3255ba6b1833c41aa654a8c6056b6 /gold/powerpc.cc | |
| parent | 9d1f51209437ebee8f0a930b9613d59759ea8dc0 (diff) | |
| download | binutils-gdb-9215b98bb27c071386a277f5578dbb17569a1471.tar.gz | |
gold --emit-relocs
A symbol value in an ELF final linked binary is absolute, in contrast
to a relocatable object file where the value is section relative. For
--emit-relocs it is therefore incorrect to use the value of a section
symbol as the addend when adjusting relocs against input section
symbols to output section symbols.
PR gold/18846
* target-reloc.h (relocate_relocs <RELOC_ADJUST_FOR_SECTION_RELA>):
Subtract os->address() from addend.
* powerpc.cc (relocate_relocs): Likewise.
Diffstat (limited to 'gold/powerpc.cc')
| -rw-r--r-- | gold/powerpc.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/gold/powerpc.cc b/gold/powerpc.cc index 540e1977c62..3311a1d5126 100644 --- a/gold/powerpc.cc +++ b/gold/powerpc.cc @@ -8110,6 +8110,7 @@ Target_powerpc<size, big_endian>::relocate_relocs( } // Get the new symbol index. + Output_section* os = NULL; if (r_sym < local_count) { switch (strategy) @@ -8134,7 +8135,7 @@ Target_powerpc<size, big_endian>::relocate_relocs( unsigned int shndx = object->local_symbol_input_shndx(r_sym, &is_ordinary); gold_assert(is_ordinary); - Output_section* os = object->output_section(shndx); + os = object->output_section(shndx); gold_assert(os != NULL); gold_assert(os->needs_symtab_index()); r_sym = os->symtab_index(); @@ -8187,7 +8188,8 @@ Target_powerpc<size, big_endian>::relocate_relocs( else if (strategy == Relocatable_relocs::RELOC_ADJUST_FOR_SECTION_RELA) { const Symbol_value<size>* psymval = object->local_symbol(orig_r_sym); - addend = psymval->value(object, addend); + gold_assert(os != NULL); + addend = psymval->value(object, addend) - os->address(); } else if (strategy == Relocatable_relocs::RELOC_SPECIAL) { |