diff options
author | Nick Clifton <nickc@redhat.com> | 2014-03-06 10:57:13 +0000 |
---|---|---|
committer | Nick Clifton <nickc@redhat.com> | 2014-03-06 10:57:13 +0000 |
commit | e9847026c92f330f18863a7b9cb0aaa299345559 (patch) | |
tree | ce50694351de5b1010ad3c17f7ebe9d08581949c /bfd/elf-attrs.c | |
parent | 5fa1d40e9790ed55931263430130e69a99329be7 (diff) | |
download | fsf-binutils-gdb-e9847026c92f330f18863a7b9cb0aaa299345559.zip fsf-binutils-gdb-e9847026c92f330f18863a7b9cb0aaa299345559.tar.gz fsf-binutils-gdb-e9847026c92f330f18863a7b9cb0aaa299345559.tar.bz2 |
Patch for PR binutils/16664 which triggers a seg-fault when attempting to
display the contents of a corrupt attribute section.
* readelf.c (process_attributes): Add checks for corrupt
attribute section names.
* elf-attrs.c (_bfd_elf_parse_attributes): Add checks for corrupt
attribute section names.
Diffstat (limited to 'bfd/elf-attrs.c')
-rw-r--r-- | bfd/elf-attrs.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/bfd/elf-attrs.c b/bfd/elf-attrs.c index d2ef769..cd0cbca 100644 --- a/bfd/elf-attrs.c +++ b/bfd/elf-attrs.c @@ -449,7 +449,7 @@ _bfd_elf_parse_attributes (bfd *abfd, Elf_Internal_Shdr * hdr) len = hdr->sh_size - 1; while (len > 0) { - int namelen; + unsigned namelen; bfd_vma section_len; int vendor; @@ -458,8 +458,11 @@ _bfd_elf_parse_attributes (bfd *abfd, Elf_Internal_Shdr * hdr) if (section_len > len) section_len = len; len -= section_len; - namelen = strlen ((char *) p) + 1; - section_len -= namelen + 4; + section_len -= 4; + namelen = strnlen ((char *) p, section_len) + 1; + if (namelen == 0 || namelen >= section_len) + break; + section_len -= namelen; if (std_sec && strcmp ((char *) p, std_sec) == 0) vendor = OBJ_ATTR_PROC; else if (strcmp ((char *) p, "gnu") == 0) |