diff options
author | Nick Clifton <nickc@redhat.com> | 2021-01-14 09:15:13 +0000 |
---|---|---|
committer | Nick Clifton <nickc@redhat.com> | 2021-01-14 09:15:13 +0000 |
commit | 7e0d77ef5ffa7e47655db2a5df5082004f7c104e (patch) | |
tree | 470243151515f6297d8b51b59b8ca28cecf0a2a5 | |
parent | 271bea6acd3759db96b33a30f110ee5e1f8146fc (diff) | |
download | fsf-binutils-gdb-7e0d77ef5ffa7e47655db2a5df5082004f7c104e.zip fsf-binutils-gdb-7e0d77ef5ffa7e47655db2a5df5082004f7c104e.tar.gz fsf-binutils-gdb-7e0d77ef5ffa7e47655db2a5df5082004f7c104e.tar.bz2 |
Fix an illegal memory access parsing a win32pstatus note with a type of 0.
* elf.c (elfcore_grok_win32pstatus): Check for a note type of 0.
-rw-r--r-- | bfd/ChangeLog | 4 | ||||
-rw-r--r-- | bfd/elf.c | 5 |
2 files changed, 7 insertions, 2 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 330b0d7..7f1b720 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,7 @@ +2021-01-14 Nick Clifton <nickc@redhat.com> + + * elf.c (elfcore_grok_win32pstatus): Check for a note type of 0. + 2021-01-13 Alan Modra <amodra@gmail.com> * Makefile.in: Regenerate. @@ -10163,7 +10163,8 @@ elfcore_grok_win32pstatus (bfd *abfd, Elf_Internal_Note *note) type = bfd_get_32 (abfd, note->descdata); - struct { + struct + { const char *type_name; unsigned long min_size; } size_check[] = @@ -10174,7 +10175,7 @@ elfcore_grok_win32pstatus (bfd *abfd, Elf_Internal_Note *note) { "NOTE_INFO_MODULE64", 16 }, }; - if (type > (sizeof(size_check)/sizeof(size_check[0]))) + if (type == 0 || type > (sizeof(size_check)/sizeof(size_check[0]))) return TRUE; if (note->descsz < size_check[type - 1].min_size) |