PR 21412, get_reloc_section assumes .rel/.rela name for SHT_REL/RELA.
This patch fixes an assumption made by code that runs for objcopy and strip, that SHT_REL/SHR_RELA sections are always named starting with a .rel/.rela prefix. I'm also modifying the interface for elf_backend_get_reloc_section, so any backend function just needs to handle name mapping. PR 21412 * elf-bfd.h (struct elf_backend_data <get_reloc_section>): Change parameters and comment. (_bfd_elf_get_reloc_section): Delete. (_bfd_elf_plt_get_reloc_section): Declare. * elf.c (_bfd_elf_plt_get_reloc_section, elf_get_reloc_section): New functions. Don't blindly skip over assumed .rel/.rela prefix. Extracted from.. (_bfd_elf_get_reloc_section): ..here. Delete. (assign_section_numbers): Call elf_get_reloc_section. * elf64-ppc.c (elf_backend_get_reloc_section): Define. * elfxx-target.h (elf_backend_get_reloc_section): Update.
This commit is contained in:
parent
97e83a100a
commit
bce964aa6c
|
@ -1,3 +1,18 @@
|
||||||
|
2017-04-23 Alan Modra <amodra@gmail.com>
|
||||||
|
|
||||||
|
PR 21412
|
||||||
|
* elf-bfd.h (struct elf_backend_data <get_reloc_section>): Change
|
||||||
|
parameters and comment.
|
||||||
|
(_bfd_elf_get_reloc_section): Delete.
|
||||||
|
(_bfd_elf_plt_get_reloc_section): Declare.
|
||||||
|
* elf.c (_bfd_elf_plt_get_reloc_section, elf_get_reloc_section):
|
||||||
|
New functions. Don't blindly skip over assumed .rel/.rela prefix.
|
||||||
|
Extracted from..
|
||||||
|
(_bfd_elf_get_reloc_section): ..here. Delete.
|
||||||
|
(assign_section_numbers): Call elf_get_reloc_section.
|
||||||
|
* elf64-ppc.c (elf_backend_get_reloc_section): Define.
|
||||||
|
* elfxx-target.h (elf_backend_get_reloc_section): Update.
|
||||||
|
|
||||||
2017-04-23 Alan Modra <amodra@gmail.com>
|
2017-04-23 Alan Modra <amodra@gmail.com>
|
||||||
|
|
||||||
PR 21409
|
PR 21409
|
||||||
|
|
|
@ -1360,8 +1360,10 @@ struct elf_backend_data
|
||||||
bfd_size_type (*maybe_function_sym) (const asymbol *sym, asection *sec,
|
bfd_size_type (*maybe_function_sym) (const asymbol *sym, asection *sec,
|
||||||
bfd_vma *code_off);
|
bfd_vma *code_off);
|
||||||
|
|
||||||
/* Return the section which RELOC_SEC applies to. */
|
/* Given NAME, the name of a relocation section stripped of its
|
||||||
asection *(*get_reloc_section) (asection *reloc_sec);
|
.rel/.rela prefix, return the section in ABFD to which the
|
||||||
|
relocations apply. */
|
||||||
|
asection *(*get_reloc_section) (bfd *abfd, const char *name);
|
||||||
|
|
||||||
/* Called to set the sh_flags, sh_link and sh_info fields of OSECTION which
|
/* Called to set the sh_flags, sh_link and sh_info fields of OSECTION which
|
||||||
has a type >= SHT_LOOS. Returns TRUE if the fields were initialised,
|
has a type >= SHT_LOOS. Returns TRUE if the fields were initialised,
|
||||||
|
@ -2448,7 +2450,7 @@ extern bfd_boolean _bfd_elf_is_function_type (unsigned int);
|
||||||
extern bfd_size_type _bfd_elf_maybe_function_sym (const asymbol *, asection *,
|
extern bfd_size_type _bfd_elf_maybe_function_sym (const asymbol *, asection *,
|
||||||
bfd_vma *);
|
bfd_vma *);
|
||||||
|
|
||||||
extern asection *_bfd_elf_get_reloc_section (asection *);
|
extern asection *_bfd_elf_plt_get_reloc_section (bfd *, const char *);
|
||||||
|
|
||||||
extern int bfd_elf_get_default_section_type (flagword);
|
extern int bfd_elf_get_default_section_type (flagword);
|
||||||
|
|
||||||
|
|
61
bfd/elf.c
61
bfd/elf.c
|
@ -3538,17 +3538,39 @@ bfd_elf_set_group_contents (bfd *abfd, asection *sec, void *failedptrarg)
|
||||||
H_PUT_32 (abfd, sec->flags & SEC_LINK_ONCE ? GRP_COMDAT : 0, loc);
|
H_PUT_32 (abfd, sec->flags & SEC_LINK_ONCE ? GRP_COMDAT : 0, loc);
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Return the section which RELOC_SEC applies to. */
|
/* Given NAME, the name of a relocation section stripped of its
|
||||||
|
.rel/.rela prefix, return the section in ABFD to which the
|
||||||
|
relocations apply. */
|
||||||
|
|
||||||
asection *
|
asection *
|
||||||
_bfd_elf_get_reloc_section (asection *reloc_sec)
|
_bfd_elf_plt_get_reloc_section (bfd *abfd, const char *name)
|
||||||
|
{
|
||||||
|
/* If a target needs .got.plt section, relocations in rela.plt/rel.plt
|
||||||
|
section likely apply to .got.plt or .got section. */
|
||||||
|
if (get_elf_backend_data (abfd)->want_got_plt
|
||||||
|
&& strcmp (name, ".plt") == 0)
|
||||||
|
{
|
||||||
|
asection *sec;
|
||||||
|
|
||||||
|
name = ".got.plt";
|
||||||
|
sec = bfd_get_section_by_name (abfd, name);
|
||||||
|
if (sec != NULL)
|
||||||
|
return sec;
|
||||||
|
name = ".got";
|
||||||
|
}
|
||||||
|
|
||||||
|
return bfd_get_section_by_name (abfd, name);
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Return the section to which RELOC_SEC applies. */
|
||||||
|
|
||||||
|
static asection *
|
||||||
|
elf_get_reloc_section (asection *reloc_sec)
|
||||||
{
|
{
|
||||||
const char *name;
|
const char *name;
|
||||||
unsigned int type;
|
unsigned int type;
|
||||||
bfd *abfd;
|
bfd *abfd;
|
||||||
|
const struct elf_backend_data *bed;
|
||||||
if (reloc_sec == NULL)
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
type = elf_section_data (reloc_sec)->this_hdr.sh_type;
|
type = elf_section_data (reloc_sec)->this_hdr.sh_type;
|
||||||
if (type != SHT_REL && type != SHT_RELA)
|
if (type != SHT_REL && type != SHT_RELA)
|
||||||
|
@ -3556,28 +3578,15 @@ _bfd_elf_get_reloc_section (asection *reloc_sec)
|
||||||
|
|
||||||
/* We look up the section the relocs apply to by name. */
|
/* We look up the section the relocs apply to by name. */
|
||||||
name = reloc_sec->name;
|
name = reloc_sec->name;
|
||||||
if (type == SHT_REL)
|
if (strncmp (name, ".rel", 4) != 0)
|
||||||
name += 4;
|
return NULL;
|
||||||
else
|
name += 4;
|
||||||
name += 5;
|
if (type == SHT_RELA && *name++ != 'a')
|
||||||
|
return NULL;
|
||||||
|
|
||||||
/* If a target needs .got.plt section, relocations in rela.plt/rel.plt
|
|
||||||
section apply to .got.plt section. */
|
|
||||||
abfd = reloc_sec->owner;
|
abfd = reloc_sec->owner;
|
||||||
if (get_elf_backend_data (abfd)->want_got_plt
|
bed = get_elf_backend_data (abfd);
|
||||||
&& strcmp (name, ".plt") == 0)
|
return bed->get_reloc_section (abfd, name);
|
||||||
{
|
|
||||||
/* .got.plt is a linker created input section. It may be mapped
|
|
||||||
to some other output section. Try two likely sections. */
|
|
||||||
name = ".got.plt";
|
|
||||||
reloc_sec = bfd_get_section_by_name (abfd, name);
|
|
||||||
if (reloc_sec != NULL)
|
|
||||||
return reloc_sec;
|
|
||||||
name = ".got";
|
|
||||||
}
|
|
||||||
|
|
||||||
reloc_sec = bfd_get_section_by_name (abfd, name);
|
|
||||||
return reloc_sec;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Assign all ELF section numbers. The dummy first section is handled here
|
/* Assign all ELF section numbers. The dummy first section is handled here
|
||||||
|
@ -3841,7 +3850,7 @@ assign_section_numbers (bfd *abfd, struct bfd_link_info *link_info)
|
||||||
if (s != NULL)
|
if (s != NULL)
|
||||||
d->this_hdr.sh_link = elf_section_data (s)->this_idx;
|
d->this_hdr.sh_link = elf_section_data (s)->this_idx;
|
||||||
|
|
||||||
s = get_elf_backend_data (abfd)->get_reloc_section (sec);
|
s = elf_get_reloc_section (sec);
|
||||||
if (s != NULL)
|
if (s != NULL)
|
||||||
{
|
{
|
||||||
d->this_hdr.sh_info = elf_section_data (s)->this_idx;
|
d->this_hdr.sh_info = elf_section_data (s)->this_idx;
|
||||||
|
|
|
@ -121,6 +121,7 @@ static bfd_vma opd_entry_value
|
||||||
#define elf_backend_special_sections ppc64_elf_special_sections
|
#define elf_backend_special_sections ppc64_elf_special_sections
|
||||||
#define elf_backend_merge_symbol_attribute ppc64_elf_merge_symbol_attribute
|
#define elf_backend_merge_symbol_attribute ppc64_elf_merge_symbol_attribute
|
||||||
#define elf_backend_merge_symbol ppc64_elf_merge_symbol
|
#define elf_backend_merge_symbol ppc64_elf_merge_symbol
|
||||||
|
#define elf_backend_get_reloc_section bfd_get_section_by_name
|
||||||
|
|
||||||
/* The name of the dynamic interpreter. This is put in the .interp
|
/* The name of the dynamic interpreter. This is put in the .interp
|
||||||
section. */
|
section. */
|
||||||
|
|
|
@ -715,7 +715,7 @@
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#ifndef elf_backend_get_reloc_section
|
#ifndef elf_backend_get_reloc_section
|
||||||
#define elf_backend_get_reloc_section _bfd_elf_get_reloc_section
|
#define elf_backend_get_reloc_section _bfd_elf_plt_get_reloc_section
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#ifndef elf_backend_copy_special_section_fields
|
#ifndef elf_backend_copy_special_section_fields
|
||||||
|
|
Loading…
Reference in New Issue