aboutsummaryrefslogtreecommitdiff
path: root/bfd/elflink.c
diff options
context:
space:
mode:
authorAlan Modra <amodra@gmail.com>2017-08-31 12:48:37 +0930
committerAlan Modra <amodra@gmail.com>2017-08-31 13:18:16 +0930
commitaf471f828cc74d65b50a7531ba2c69522266cfe9 (patch)
treef9a11fc0d22c56b73b084b2e111efa198667d55c /bfd/elflink.c
parent3321e42bf2b5819b76ca36e8964fe7cf9a8ae9a2 (diff)
downloadgdb-af471f828cc74d65b50a7531ba2c69522266cfe9.zip
gdb-af471f828cc74d65b50a7531ba2c69522266cfe9.tar.gz
gdb-af471f828cc74d65b50a7531ba2c69522266cfe9.tar.bz2
PR22048, Incorrect .eh_frame section in libc.so
PR 21441 PR 22048 * elflink.c (bfd_elf_discard_info): Don't pad embedded zero terminators.
Diffstat (limited to 'bfd/elflink.c')
-rw-r--r--bfd/elflink.c25
1 files changed, 15 insertions, 10 deletions
diff --git a/bfd/elflink.c b/bfd/elflink.c
index ceacb37..f6f57fa 100644
--- a/bfd/elflink.c
+++ b/bfd/elflink.c
@@ -13878,17 +13878,22 @@ bfd_elf_discard_info (bfd *output_bfd, struct bfd_link_info *info)
/* Any prior sections must pad the last FDE out to the
output section alignment. Otherwise we might have zero
padding between sections, which would be seen as a
- terminator. */
+ terminator. If there is a terminator in the middle of
+ FDEs, don't increase its size as that will write bogus
+ data of whatever was after the terminator in the input
+ file, to the output file. */
for (; i != NULL; i = i->map_tail.s)
- {
- bfd_size_type size = (i->size + eh_alignment - 1) & -eh_alignment;
- if (i->size != size)
- {
- i->size = size;
- changed = 1;
- eh_changed = 1;
- }
- }
+ if (i->size != 4)
+ {
+ bfd_size_type size
+ = (i->size + eh_alignment - 1) & -eh_alignment;
+ if (i->size != size)
+ {
+ i->size = size;
+ changed = 1;
+ eh_changed = 1;
+ }
+ }
}
if (eh_changed)
elf_link_hash_traverse (elf_hash_table (info),