]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
aarch64: bugfix when finding 1st bfd input with GNU property
authorMatthieu Longo <matthieu.longo@arm.com>
Wed, 6 Nov 2024 17:59:46 +0000 (17:59 +0000)
committerMatthieu Longo <matthieu.longo@arm.com>
Mon, 2 Dec 2024 15:18:40 +0000 (15:18 +0000)
The current implementation of searching the first input BFD with GNU
properties has a bug. The search was not filtering on object inputs
belonging to the output link unit only, but was also including dynamic
objects, BFD plugins, and linker-created files.
This means that the initial initialization of the output properties
were skewed, and warnings on input files that should have been emitted
were not.

This patch fixes the filtering to exclude the object input files not
belonging to the output link unit, not having the same ELF class, and
not the same target architecture.

bfd/elfxx-aarch64.c

index 896497a7812166ec0a0f1fc7f6ace9c74002c5b3..c01e0ab6599bc4fa02b9c69471d0bed86736eec7 100644 (file)
@@ -707,11 +707,16 @@ _bfd_aarch64_elf_find_1st_bfd_input_with_gnu_property (
   bool *has_gnu_property)
 {
   BFD_ASSERT (has_gnu_property);
+  const struct elf_backend_data *obfd = get_elf_backend_data (info->output_bfd);
   bfd *pbfd = info->input_bfds;
   bfd *prev = NULL;
   for (; pbfd != NULL; pbfd = pbfd->link.next)
     if (bfd_get_flavour (pbfd) == bfd_target_elf_flavour
-       && bfd_count_sections (pbfd) != 0)
+       && bfd_count_sections (pbfd) != 0
+       && (pbfd->flags & (DYNAMIC | BFD_PLUGIN | BFD_LINKER_CREATED)) == 0
+       && (obfd->elf_machine_code
+           == get_elf_backend_data (pbfd)->elf_machine_code)
+       && (obfd->s->elfclass == get_elf_backend_data (pbfd)->s->elfclass))
       {
        /* Does the input have a list of GNU properties ? */
        if (elf_properties (pbfd) != NULL)