diff options
author | Alan Modra <amodra@gmail.com> | 2022-11-14 08:25:16 +1030 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2022-11-14 11:36:46 +1030 |
commit | 4f5c4fce8878830d8e4e928fa0dbe3f4ee2eb2cd (patch) | |
tree | 85d0c7efd9d38d9fb3caf11b5c1ee0e9e754183a /binutils/objcopy.c | |
parent | f31e4e02eed3a758d757c7ede855b7aec1dd3850 (diff) | |
download | binutils-4f5c4fce8878830d8e4e928fa0dbe3f4ee2eb2cd.zip binutils-4f5c4fce8878830d8e4e928fa0dbe3f4ee2eb2cd.tar.gz binutils-4f5c4fce8878830d8e4e928fa0dbe3f4ee2eb2cd.tar.bz2 |
objcopy renaming section with explicit flags
This tidies SEC_RELOC handling in bfd, in the process fixing a bug
with objcopy when renaming sections.
bfd/
* reloc.c (_bfd_generic_set_reloc): Set/clear SEC_RELOC depending
on reloc count.
* elf64-sparc.c (elf64_sparc_set_reloc): Likewise.
binutils/
* objcopy.c (copy_relocations_in_section): Remove now unnecessary
clearing of SEC_RELOC.
* testsuite/binutils-all/rename-section-01.d: New test.
* testsuite/binutils-all/objcopy.exp: Run it.
gas/
* write.c (size_seg): Remove unneccesary twiddle of SEC_RELOC.
(write_relocs): Likewise. Always call bfd_set_reloc.
Diffstat (limited to 'binutils/objcopy.c')
-rw-r--r-- | binutils/objcopy.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/binutils/objcopy.c b/binutils/objcopy.c index d886e3a..3d88624 100644 --- a/binutils/objcopy.c +++ b/binutils/objcopy.c @@ -4331,10 +4331,7 @@ copy_relocations_in_section (bfd *ibfd, sec_ptr isection, void *obfdarg) } if (relsize == 0) - { - bfd_set_reloc (obfd, osection, NULL, 0); - osection->flags &= ~SEC_RELOC; - } + bfd_set_reloc (obfd, osection, NULL, 0); else { if (isection->orelocation != NULL) @@ -4377,8 +4374,6 @@ copy_relocations_in_section (bfd *ibfd, sec_ptr isection, void *obfdarg) } bfd_set_reloc (obfd, osection, relcount == 0 ? NULL : relpp, relcount); - if (relcount == 0) - osection->flags &= ~SEC_RELOC; } } |