|
| 1 | +From 82732e362aaf9eef36cdcabe4330b5304fa1ef8d Mon Sep 17 00:00:00 2001 |
| 2 | +From: AllSpark < [email protected]> |
| 3 | +Date: Mon, 27 Oct 2025 09:26:13 +0000 |
| 4 | +Subject: [PATCH] elf: Return error on unsorted symbol table if not allowed |
| 5 | + |
| 6 | +Normally ELF symbol table should be sorted, i.e., local symbols precede |
| 7 | +global symbols. Irix 6 is an exception and its elf_bad_symtab is set |
| 8 | +to true. Issue an error if elf_bad_symtab is false and symbol table is |
| 9 | +unsorted. |
| 10 | + |
| 11 | +\tPR ld/33450 |
| 12 | +\t* elflink.c (set_symbol_value): Change return type to bool and |
| 13 | +\treturn false on error. Issue an error on unsorted symbol table |
| 14 | +\tif not allowed. |
| 15 | +\t(elf_link_input_bfd): Return false if set_symbol_value reurns |
| 16 | +\tfalse. |
| 17 | + |
| 18 | +Signed-off-by: H.J. Lu < [email protected]> |
| 19 | +Signed-off-by: Azure Linux Security Servicing Account < [email protected]> |
| 20 | +Upstream-reference: AI Backport of https://github.com/bminor/binutils-gdb/commit/aeaaa9af6359c8e394ce9cf24911fec4f4d23703.patch |
| 21 | +--- |
| 22 | + bfd/elflink.c | 19 ++++++++++++++----- |
| 23 | + 1 file changed, 14 insertions(+), 5 deletions(-) |
| 24 | + |
| 25 | +diff --git a/bfd/elflink.c b/bfd/elflink.c |
| 26 | +index dc0267e..1a1a44e 100644 |
| 27 | +--- a/bfd/elflink.c |
| 28 | ++++ b/bfd/elflink.c |
| 29 | +@@ -8596,7 +8596,7 @@ struct elf_outext_info |
| 30 | + <binary-operator> := as in C |
| 31 | + <unary-operator> := as in C, plus "0-" for unambiguous negation. */ |
| 32 | + |
| 33 | +-static void |
| 34 | ++static bool |
| 35 | + set_symbol_value (bfd *bfd_with_globals, |
| 36 | + Elf_Internal_Sym *isymbuf, |
| 37 | + size_t locsymcount, |
| 38 | +@@ -8618,9 +8618,15 @@ set_symbol_value (bfd *bfd_with_globals, |
| 39 | + "absolute" section and give it a value. */ |
| 40 | + sym->st_shndx = SHN_ABS; |
| 41 | + sym->st_value = val; |
| 42 | +- return; |
| 43 | ++ return true; |
| 44 | ++ } |
| 45 | ++ if (!elf_bad_symtab (bfd_with_globals)) |
| 46 | ++ { |
| 47 | ++ _bfd_error_handler (_("%pB: corrupt symbol table"), |
| 48 | ++ bfd_with_globals); |
| 49 | ++ bfd_set_error (bfd_error_bad_value); |
| 50 | ++ return false; |
| 51 | + } |
| 52 | +- BFD_ASSERT (elf_bad_symtab (bfd_with_globals)); |
| 53 | + extsymoff = 0; |
| 54 | + } |
| 55 | + |
| 56 | +@@ -8635,6 +8641,7 @@ set_symbol_value (bfd *bfd_with_globals, |
| 57 | + h->root.type = bfd_link_hash_defined; |
| 58 | + h->root.u.def.value = val; |
| 59 | + h->root.u.def.section = bfd_abs_section_ptr; |
| 60 | ++ return true; |
| 61 | + } |
| 62 | + |
| 63 | + static bool |
| 64 | +@@ -11338,8 +11345,10 @@ elf_link_input_bfd (struct elf_final_link_info *flinfo, bfd *input_bfd) |
| 65 | + return false; |
| 66 | + |
| 67 | + /* Symbol evaluated OK. Update to absolute value. */ |
| 68 | +- set_symbol_value (input_bfd, isymbuf, locsymcount, |
| 69 | +- r_symndx, val); |
| 70 | ++ if (!set_symbol_value (input_bfd, isymbuf, locsymcount, r_symndx, |
| 71 | ++ val)) |
| 72 | ++ return false; |
| 73 | ++ |
| 74 | + continue; |
| 75 | + } |
| 76 | + |
| 77 | +-- |
| 78 | +2.45.4 |
| 79 | + |
0 commit comments