summaryrefslogtreecommitdiff
path: root/bfd
diff options
context:
space:
mode:
authorThomas Preud'homme <thomas.preudhomme@arm.com>2016-09-14 16:02:17 +0100
committerThomas Preud'homme <thomas.preudhomme@arm.com>2016-09-14 16:03:43 +0100
commit4f90d84b2f2995829d6af475077598d45ef1d127 (patch)
treee745bdc7c37b3878e45e4a333f22ab397570ea5a /bfd
parente6d042fe27102cb789407ccb2ec1663aa9c65129 (diff)
downloadbinutils-gdb-4f90d84b2f2995829d6af475077598d45ef1d127.tar.gz
Fix ld --gc-section segfault with ARMv8-M entry function in absolute section
bfd/ 2016-09-14 Thomas Preud'homme <thomas.preudhomme@arm.com> * elf32-arm.c (elf32_arm_gc_mark_extra_sections): Only mark section not already marked. ld/ 2016-09-14 Thomas Preud'homme <thomas.preudhomme@arm.com> * testsuite/ld-arm/cmse-veneers.s: Add a test for ARMv8-M Security Extensions entry functions in absolute section. * testsuite/ld-arm/cmse-veneers.rd: Adapt expected output accordingly.
Diffstat (limited to 'bfd')
-rw-r--r--bfd/ChangeLog5
-rw-r--r--bfd/elf32-arm.c3
2 files changed, 7 insertions, 1 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 567e18f195b..b54d0c938ea 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2016-09-14 Thomas Preud'homme <thomas.preudhomme@arm.com>
+
+ * elf32-arm.c (elf32_arm_gc_mark_extra_sections): Only mark section
+ not already marked.
+
2016-09-14 Nick Clifton <nickc@redhat.com>
PR binutils/20605
diff --git a/bfd/elf32-arm.c b/bfd/elf32-arm.c
index 5275caef164..6e68be1c09d 100644
--- a/bfd/elf32-arm.c
+++ b/bfd/elf32-arm.c
@@ -14906,7 +14906,8 @@ elf32_arm_gc_mark_extra_sections (struct bfd_link_info *info,
if (ARM_GET_SYM_CMSE_SPCL (cmse_hash->root.target_internal))
{
cmse_sec = cmse_hash->root.root.u.def.section;
- if (!_bfd_elf_gc_mark (info, cmse_sec, gc_mark_hook))
+ if (!cmse_sec->gc_mark &&
+ !_bfd_elf_gc_mark (info, cmse_sec, gc_mark_hook))
return FALSE;
}
}