Commit 23a4ca4e749a300a29abff1e304c55b55b6e59a3

Authored by Josh Poimboeuf
Committed by Greg Kroah-Hartman
1 parent 3e04e09855

objtool: Warn on stripped section symbol

commit 830c1e3d16b2c1733cd1ec9c8f4d47a398ae31bc

With the following fix:

  2a0098d70640 ("objtool: Fix seg fault with gold linker")

... a seg fault was avoided, but the original seg fault condition in
objtool wasn't fixed.  Replace the seg fault with an error message.

Suggested-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Josh Poimboeuf <jpoimboe@redhat.com>
Cc: Andy Lutomirski <luto@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Guenter Roeck <linux@roeck-us.net>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Juergen Gross <jgross@suse.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/dc4585a70d6b975c99fc51d1957ccdde7bd52f3a.1517284349.git.jpoimboe@redhat.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Showing 1 changed file with 5 additions and 0 deletions Side-by-side Diff

tools/objtool/orc_gen.c
... ... @@ -98,6 +98,11 @@
98 98 struct orc_entry *orc;
99 99 struct rela *rela;
100 100  
  101 + if (!insn_sec->sym) {
  102 + WARN("missing symbol for section %s", insn_sec->name);
  103 + return -1;
  104 + }
  105 +
101 106 /* populate ORC data */
102 107 orc = (struct orc_entry *)u_sec->data->d_buf + idx;
103 108 memcpy(orc, o, sizeof(*orc));