diff options
author | Alan Modra <amodra@gmail.com> | 2017-10-01 12:07:07 +1030 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2017-10-01 12:07:07 +1030 |
commit | 4f1881b94473f1034f950feb863b464435a8fb5f (patch) | |
tree | ad86f5575996ed09dbe71614dc368a1e2936a358 /binutils/objdump.c | |
parent | 3678b25d0c5fee04a91ad90ed2965513ba0ca11f (diff) |
PR22230, buffer overflow in display_debug_macro
PR 22230
* objdump.c (load_specific_debug_section): Allocate an extra byte
for a terminating NUL.
Diffstat (limited to 'binutils/objdump.c')
-rw-r--r-- | binutils/objdump.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/binutils/objdump.c b/binutils/objdump.c index 3b2c7a3878..83b8b2a273 100644 --- a/binutils/objdump.c +++ b/binutils/objdump.c @@ -2475,17 +2475,18 @@ load_specific_debug_section (enum dwarf_section_display_enum debug, section->num_relocs = 0; section->address = bfd_get_section_vma (abfd, sec); section->size = bfd_get_section_size (sec); - section->start = NULL; + section->start = malloc (section->size + 1); section->user_data = sec; - ret = bfd_get_full_section_contents (abfd, sec, §ion->start); - - if (! ret) + if (section->start == NULL + || !bfd_get_full_section_contents (abfd, sec, §ion->start)) { free_debug_section (debug); printf (_("\nCan't get contents for section '%s'.\n"), section->name); return 0; } + /* Ensure any string section has a terminating NUL. */ + section->start[section->size] = 0; if (is_relocatable && debug_displays [debug].relocate) { |