diff options
| author | Alan Modra <amodra@gmail.com> | 2014-12-03 21:56:19 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2014-12-04 01:09:55 +1030 |
| commit | 1f98a0748c4dfae57a76c408501c938480c1173c (patch) | |
| tree | 647e93f222ec51a17757228f01c0ab294420919c /gold | |
| parent | ff1eb2b544c8500f48eff3418ac1e005acd5e3fe (diff) | |
| download | fsf-binutils-gdb-1f98a0748c4dfae57a76c408501c938480c1173c.zip fsf-binutils-gdb-1f98a0748c4dfae57a76c408501c938480c1173c.tar.gz fsf-binutils-gdb-1f98a0748c4dfae57a76c408501c938480c1173c.tar.bz2 | |
powerpc gold, fix pr17566
Code stolen from arm.cc.
PR 17566
* powerpc.cc (Target_powerpc::Scan::local): Use add_local_section
when adding dynamic relocations against section symbols.
Diffstat (limited to 'gold')
| -rw-r--r-- | gold/ChangeLog | 6 | ||||
| -rw-r--r-- | gold/powerpc.cc | 20 |
2 files changed, 23 insertions, 3 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog index 8bd1afc..b75530f 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,9 @@ +2014-12-03 Alan Modra <amodra@gmail.com> + + PR 17566 + * powerpc.cc (Target_powerpc::Scan::local): Use add_local_section + when adding dynamic relocations against section symbols. + 2014-12-01 Dimitry Ivanov <dimitry@google.com> * layout.cc (Layout::finish_dynamic_section): When '-z global' diff --git a/gold/powerpc.cc b/gold/powerpc.cc index 554bf38..5805c0bd 100644 --- a/gold/powerpc.cc +++ b/gold/powerpc.cc @@ -5566,10 +5566,10 @@ Target_powerpc<size, big_endian>::Scan::local( { Reloc_section* rela_dyn = target->rela_dyn_section(symtab, layout, is_ifunc); + unsigned int r_sym = elfcpp::elf_r_sym<size>(reloc.get_r_info()); if ((size == 32 && r_type == elfcpp::R_POWERPC_ADDR32) || (size == 64 && r_type == elfcpp::R_PPC64_ADDR64)) { - unsigned int r_sym = elfcpp::elf_r_sym<size>(reloc.get_r_info()); unsigned int dynrel = (is_ifunc ? elfcpp::R_POWERPC_IRELATIVE : elfcpp::R_POWERPC_RELATIVE); rela_dyn->add_local_relative(object, r_sym, dynrel, @@ -5577,14 +5577,28 @@ Target_powerpc<size, big_endian>::Scan::local( reloc.get_r_offset(), reloc.get_r_addend(), false); } - else + else if (lsym.get_st_type() != elfcpp::STT_SECTION) { check_non_pic(object, r_type); - unsigned int r_sym = elfcpp::elf_r_sym<size>(reloc.get_r_info()); rela_dyn->add_local(object, r_sym, r_type, output_section, data_shndx, reloc.get_r_offset(), reloc.get_r_addend()); } + else + { + gold_assert(lsym.get_st_value() == 0); + unsigned int shndx = lsym.get_st_shndx(); + bool is_ordinary; + shndx = object->adjust_sym_shndx(r_sym, shndx, + &is_ordinary); + if (!is_ordinary) + object->error(_("section symbol %u has bad shndx %u"), + r_sym, shndx); + else + rela_dyn->add_local_section(object, shndx, r_type, + output_section, data_shndx, + reloc.get_r_offset()); + } } break; |