From 8e4979ac1ea78147ecbcbf81af5e946873dda079 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Thu, 16 Apr 2020 18:02:10 +0100 Subject: [PATCH] Stop the MIPS assembler from accepting ifunc symbols. PR 25803 gas * config/obj-elf.c (obj_elf_type): Reject ifunc symbols on MIPS targets. * testsuite/gas/elf/elf.exp: Add MIPS targets to the list to skip for the type-2 test. * testsuite/gas/elf/type-noifunc.e: Update to allow for MIPS targets running this test. bfd * elfxx-mips.c (_bfd_mips_elf_adjust_dynamic_symbol): Replace an abort with a more helpful error message. --- bfd/ChangeLog | 6 ++++++ bfd/elfxx-mips.c | 21 +++++++++++++++------ gas/ChangeLog | 11 +++++++++++ gas/config/obj-elf.c | 4 ++++ gas/testsuite/gas/elf/elf.exp | 5 +++-- gas/testsuite/gas/elf/type-noifunc.e | 2 +- 6 files changed, 40 insertions(+), 9 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index bb6b9f2261c..7f361d7fb18 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2020-04-16 Nick Clifton + + PR 25803 + * elfxx-mips.c (_bfd_mips_elf_adjust_dynamic_symbol): Replace an + abort with a more helpful error message. + 2020-04-16 Alan Modra PR 25827 diff --git a/bfd/elfxx-mips.c b/bfd/elfxx-mips.c index ae8478270ef..2f26d7ae832 100644 --- a/bfd/elfxx-mips.c +++ b/bfd/elfxx-mips.c @@ -9311,12 +9311,21 @@ _bfd_mips_elf_adjust_dynamic_symbol (struct bfd_link_info *info, hmips = (struct mips_elf_link_hash_entry *) h; /* Make sure we know what is going on here. */ - BFD_ASSERT (dynobj != NULL - && (h->needs_plt - || h->is_weakalias - || (h->def_dynamic - && h->ref_regular - && !h->def_regular))); + if (dynobj == NULL + || (! h->needs_plt + && ! h->is_weakalias + && (! h->def_dynamic + || ! h->ref_regular + || h->def_regular))) + { + if (h->type == STT_GNU_IFUNC) + _bfd_error_handler (_("IFUNC symbol %s in dynamic symbol table - IFUNCS are not supported"), + h->root.root.string); + else + _bfd_error_handler (_("non-dynamic symbol %s in dynamic symbol table"), + h->root.root.string); + return TRUE; + } hmips = (struct mips_elf_link_hash_entry *) h; diff --git a/gas/ChangeLog b/gas/ChangeLog index 09ad599c5c9..e0c51ebcc5a 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,14 @@ +2020-04-16 Gagan Singh Sidhu + Nick Clifton + + PR 25803 + * config/obj-elf.c (obj_elf_type): Reject ifunc symbols on MIPS + targets. + * testsuite/gas/elf/elf.exp: Add MIPS targets to the list to skip + for the type-2 test. + * testsuite/gas/elf/type-noifunc.e: Update to allow for MIPS + targets running this test. + 2020-02-16 David Faust * testsuite/gas/bpf/bpf.exp: Run jump32 tests. diff --git a/gas/config/obj-elf.c b/gas/config/obj-elf.c index a6dcdaf4a7e..5e7e8f08779 100644 --- a/gas/config/obj-elf.c +++ b/gas/config/obj-elf.c @@ -2177,6 +2177,10 @@ obj_elf_type (int ignore ATTRIBUTE_UNUSED) && bed->elf_osabi != ELFOSABI_FREEBSD) as_bad (_("symbol type \"%s\" is supported only by GNU " "and FreeBSD targets"), type_name); + /* MIPS targets do not support IFUNCS. */ + else if (bed->target_id == MIPS_ELF_DATA) + as_bad (_("symbol type \"%s\" is not supported by " + "MIPS targets"), type_name); elf_tdata (stdoutput)->has_gnu_osabi |= elf_gnu_osabi_ifunc; type = BSF_FUNCTION | BSF_GNU_INDIRECT_FUNCTION; } diff --git a/gas/testsuite/gas/elf/elf.exp b/gas/testsuite/gas/elf/elf.exp index c80175cc51d..31026e4148e 100644 --- a/gas/testsuite/gas/elf/elf.exp +++ b/gas/testsuite/gas/elf/elf.exp @@ -220,8 +220,9 @@ if { [is_elf_format] } then { # in the symbol type test. # We also need to exclude targets that do not support unique objects. if { [istarget "*-*-hpux*"] - || [istarget "arm*-*-*"] - || [istarget "msp*-*-*"] + || [istarget "arm*-*-*"] + || [istarget "mips*-*-*"] + || [istarget "msp*-*-*"] || [istarget "visium-*-*"] || ![supports_gnu_unique] } then { diff --git a/gas/testsuite/gas/elf/type-noifunc.e b/gas/testsuite/gas/elf/type-noifunc.e index ddeadd7d3a2..76a42d09b24 100644 --- a/gas/testsuite/gas/elf/type-noifunc.e +++ b/gas/testsuite/gas/elf/type-noifunc.e @@ -1,5 +1,5 @@ +.: 0+0 +1 +FUNC +LOCAL +DEFAULT +. function +.: 0+0 +1 +OBJECT +LOCAL +DEFAULT +. object - +.: 0+1 +1 +TLS +LOCAL +DEFAULT +. tls_object + +[0-9]+: 0+1 +1 +TLS +LOCAL +DEFAULT +. tls_object +..: 0+2 +1 +NOTYPE +LOCAL +DEFAULT +. notype +..: 0+1 +1 +(COMMON|OBJECT) +GLOBAL +DEFAULT +(ANSI_|)COM common -- 2.30.2