From a78c3c9717e8fa98b11482f948e71c6d1d9d0e44 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Fri, 29 Sep 2023 15:24:26 +0100 Subject: Fix: Segmentation fault caused by npd in objdump PR 30906 * elf.c (_bfd_elf_slurp_version_tables): Test that the verref section header has been initialised before using it. --- bfd/elf.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'bfd/elf.c') diff --git a/bfd/elf.c b/bfd/elf.c index 2bd236b..d5c570f 100644 --- a/bfd/elf.c +++ b/bfd/elf.c @@ -9482,7 +9482,9 @@ _bfd_elf_slurp_version_tables (bfd *abfd, bool default_imported_symver) = elf_tdata (abfd)->dt_strtab + iverneed->vn_file; else iverneed->vn_filename = NULL; - } + } + else if (hdr == NULL) + goto error_return_bad_verref; else iverneed->vn_filename = bfd_elf_string_from_elf_section (abfd, hdr->sh_link, @@ -9525,6 +9527,8 @@ _bfd_elf_slurp_version_tables (bfd *abfd, bool default_imported_symver) else ivernaux->vna_nodename = NULL; } + else if (hdr == NULL) + goto error_return_bad_verref; else ivernaux->vna_nodename = bfd_elf_string_from_elf_section (abfd, hdr->sh_link, @@ -9555,7 +9559,7 @@ _bfd_elf_slurp_version_tables (bfd *abfd, bool default_imported_symver) iverneed->vn_nextref = NULL; if (iverneed->vn_next == 0) break; - if (i + 1 < hdr->sh_info) + if (hdr != NULL && (i + 1 < hdr->sh_info)) iverneed->vn_nextref = iverneed + 1; if (iverneed->vn_next -- cgit v1.1