diff options
| author | Alan Modra <amodra@gmail.com> | 2018-12-28 09:41:44 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2018-12-28 15:02:08 +1030 |
| commit | 0e41bebb938dbe9eae7063f5321429194bfc5ab7 (patch) | |
| tree | b29f3fb4479bd29ebb404d81e1d0e4f7d1f2fd75 /bfd/linker.c | |
| parent | cb87d9f1a49986b2eb92ac381444f4cc7c9d8a4f (diff) | |
| download | fsf-binutils-gdb-0e41bebb938dbe9eae7063f5321429194bfc5ab7.zip fsf-binutils-gdb-0e41bebb938dbe9eae7063f5321429194bfc5ab7.tar.gz fsf-binutils-gdb-0e41bebb938dbe9eae7063f5321429194bfc5ab7.tar.bz2 | |
PR24015, glibc-2.28 on little-endian mips32 broken
Commit 2bf2bf23da exposed a bug on targets that create common sections
other than the standard ELF SHN_COMMON. If these are output by ld -r,
then their type becomes SHT_PROGBITS unless the target handles them
specially (eg. by elf_backend_special_sections), and if they are
merged into .bss/.sbss by ld -r then that section becomes SHT_PROGBITS.
Worse, if they are output by ld -r, then their size is increased by
bfd_generic_define_common_symbol during final link, which leads to
bogus file contents being copied to output.
For mips, it seems to me that the .scommon section should not be
output for ld -r, but I haven't made that change in this patch.
PR 24015
* elf.c (bfd_elf_get_default_section_type): Make common sections
SHT_NOBITS.
* linker.c (bfd_generic_define_common_symbol): Clear
SEC_HAS_CONTENTS.
Diffstat (limited to 'bfd/linker.c')
| -rw-r--r-- | bfd/linker.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bfd/linker.c b/bfd/linker.c index 9fee90d..fff9d9b 100644 --- a/bfd/linker.c +++ b/bfd/linker.c @@ -3119,7 +3119,7 @@ bfd_generic_define_common_symbol (bfd *output_bfd, /* Make sure the section is allocated in memory, and make sure that it is no longer a common section. */ section->flags |= SEC_ALLOC; - section->flags &= ~SEC_IS_COMMON; + section->flags &= ~(SEC_IS_COMMON | SEC_HAS_CONTENTS); return TRUE; } |