elf: Return error on unsorted symbol table if not allowed

Normally ELF symbol table should be sorted, i.e., local symbols precede
global symbols.  Irix 6 is an exception and its elf_bad_symtab is set
to true.  Issue an error if elf_bad_symtab is false and symbol table is
unsorted.

	PR ld/33450
	* elflink.c (set_symbol_value): Change return type to bool and
	return false on error.  Issue an error on unsorted symbol table
	if not allowed.
	(elf_link_input_bfd): Return false if set_symbol_value reurns
	false.

Signed-off-by: H.J. Lu <hjl.tools@gmail.com>

diff --git a/bfd/elflink.c b/bfd/elflink.c
index 66982f82b94a597f2e647497a0ef2045042d2d13..54f0d6e957e9570cb7f7c78230a563326acc72fd 100644 (file)
--- a/bfd/elflink.c
+++ b/bfd/elflink.c
@@ -9155,7 +9155,7 @@ struct elf_outext_info
    <binary-operator> := as in C
    <unary-operator> := as in C, plus "0-" for unambiguous negation.  */
 
-static void
+static bool
 set_symbol_value (bfd *bfd_with_globals,
                  Elf_Internal_Sym *isymbuf,
                  size_t locsymcount,
@@ -9176,9 +9176,15 @@ set_symbol_value (bfd *bfd_with_globals,
             "absolute" section and give it a value.  */
          sym->st_shndx = SHN_ABS;
          sym->st_value = val;
-         return;
+         return true;
+       }
+      if (!elf_bad_symtab (bfd_with_globals))
+       {
+         _bfd_error_handler (_("%pB: corrupt symbol table"),
+                             bfd_with_globals);
+         bfd_set_error (bfd_error_bad_value);
+         return false;
        }
-      BFD_ASSERT (elf_bad_symtab (bfd_with_globals));
       extsymoff = 0;
     }
 
@@ -9188,11 +9194,12 @@ set_symbol_value (bfd *bfd_with_globals,
   if (h == NULL)
     {
       /* FIXMEL What should we do ?  */
-      return;
+      return false;
     }
   h->root.type = bfd_link_hash_defined;
   h->root.u.def.value = val;
   h->root.u.def.section = bfd_abs_section_ptr;
+  return true;
 }
 
 static bool
@@ -11890,8 +11897,10 @@ elf_link_input_bfd (struct elf_final_link_info *flinfo, bfd *input_bfd)
                    return false;
 
                  /* Symbol evaluated OK.  Update to absolute value.  */
-                 set_symbol_value (input_bfd, isymbuf, locsymcount,
-                                   r_symndx, val);
+                 if (!set_symbol_value (input_bfd, isymbuf, locsymcount, r_symndx,
+                                        val))
+                   return false;
+
                  continue;
                }