From 0e41bebb938dbe9eae7063f5321429194bfc5ab7 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Fri, 28 Dec 2018 09:41:44 +1030 Subject: 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. --- bfd/linker.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'bfd/linker.c') 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; } -- cgit v1.1