diff options
| author | Nick Clifton <nickc@redhat.com> | 2014-12-09 12:42:18 +0000 |
|---|---|---|
| committer | Nick Clifton <nickc@redhat.com> | 2014-12-09 12:42:18 +0000 |
| commit | f64e188b58f4aab4cbd03aa6e9fc1aa602546e26 (patch) | |
| tree | 46d8f90891842ce15eee893ce5489835e9e65a20 /binutils/objdump.c | |
| parent | 137d1369ac054744d27f19e95aa8a739e6c0068d (diff) | |
| download | fsf-binutils-gdb-f64e188b58f4aab4cbd03aa6e9fc1aa602546e26.zip fsf-binutils-gdb-f64e188b58f4aab4cbd03aa6e9fc1aa602546e26.tar.gz fsf-binutils-gdb-f64e188b58f4aab4cbd03aa6e9fc1aa602546e26.tar.bz2 | |
More fixes for memory access violations triggered by fuzzed binaries.
PR binutils/17512
* objdump.c (display_any_bfd): Avoid infinite loop closing and
opening the same archive again and again.
* archive64.c (bfd_elf64_archive_slurp_armap): Add range checks.
* libbfd.c (safe_read_leb128): New function.
* libbfd-in.h (safe_read_leb128): Add prototype.
* libbfd.h: Regenerate.
* elf-attrs.c (_bfd_elf_parse_attributes): Use safe_read_leb128.
Check for an over-long subsection length.
* elf.c (elf_parse_notes): Check that the namedata is long enough
for the string comparison that is about to be performed.
(elf_read_notes): Zero-terminate the note buffer.
Diffstat (limited to 'binutils/objdump.c')
| -rw-r--r-- | binutils/objdump.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/binutils/objdump.c b/binutils/objdump.c index b43d111..ee3a084 100644 --- a/binutils/objdump.c +++ b/binutils/objdump.c @@ -3426,7 +3426,15 @@ display_any_bfd (bfd *file, int level) display_any_bfd (arfile, level + 1); if (last_arfile != NULL) - bfd_close (last_arfile); + { + bfd_close (last_arfile); + /* PR 17512: file: ac585d01. */ + if (arfile == last_arfile) + { + last_arfile = NULL; + break; + } + } last_arfile = arfile; } |