diff options
| author | Alan Modra <amodra@gmail.com> | 2022-11-23 22:12:30 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2022-11-23 22:38:48 +1030 |
| commit | e6b6fad2fe4d180bcd65a1e0aabc6ba763901346 (patch) | |
| tree | bc1ceb0e32d8f6a15365185e2947bc6d724dd0e0 /binutils | |
| parent | d499fb89448c3f2e061813f12a1b63fbaa29422e (diff) | |
| download | gdb-e6b6fad2fe4d180bcd65a1e0aabc6ba763901346.zip gdb-e6b6fad2fe4d180bcd65a1e0aabc6ba763901346.tar.gz gdb-e6b6fad2fe4d180bcd65a1e0aabc6ba763901346.tar.bz2 | |
PR22509 - Null pointer dereference on coff_slurp_reloc_table
This extends the commit 4581a1c7d304 fix to more targets, which
hardens BFD a little. I think the real underlying problem was the
bfd_canonicalize_reloc call in load_specific_debug_section which
passed a NULL for "symbols". Fix that too.
PR 22509
bfd/
* aoutx.h (swap_ext_reloc_out): Gracefully handle NULL symbols.
* i386lynx.c (swap_ext_reloc_out): Likewise.
* pdp11.c (pdp11_aout_swap_reloc_out): Likewise.
* coff-tic30.c (reloc_processing): Likewise.
* coff-tic4x.c (tic4x_reloc_processing): Likewise.
* coff-tic54x.c (tic54x_reloc_processing): Likewise.
* coff-z80.c (reloc_processing): Likewise.
* coff-z8k.c (reloc_processing): Likewise.
* ecoff.c (ecoff_slurp_reloc_table): Likewise.
* som.c (som_set_reloc_info): Likewise.
binutils/
* objdump.c (load_specific_debug_section): Pass syms to
bfd_canonicalize_reloc.
Diffstat (limited to 'binutils')
| -rw-r--r-- | binutils/objdump.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/binutils/objdump.c b/binutils/objdump.c index 61a1874..9b27ce7 100644 --- a/binutils/objdump.c +++ b/binutils/objdump.c @@ -4238,7 +4238,7 @@ load_specific_debug_section (enum dwarf_section_display_enum debug, relocs = (arelent **) xmalloc (reloc_size); - reloc_count = bfd_canonicalize_reloc (abfd, sec, relocs, NULL); + reloc_count = bfd_canonicalize_reloc (abfd, sec, relocs, syms); if (reloc_count <= 0) free (relocs); else |