diff --git a/binutils/ChangeLog b/binutils/ChangeLog index d9f08991f9..d45eeaf9b5 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,9 @@ +2017-10-01 Alan Modra + + PR 22230 + * objdump.c (load_specific_debug_section): Allocate an extra byte + for a terminating NUL. + 2017-09-30 Alan Modra PR 21978 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) {