espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-07 16:14:12 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fixes a problem displaying the contents of a binary containing corrupt debug

Browse Source 
information, specifically a DW_AT_MIPS_linkage_name attribute that has a numeric
value rather than a string value.

	PR binutils/16949
	* dwarf2.c (is_str_attr): New function.
	(find_abstract_instance_name): Use it to determine when an
	attribute has a string value.
This commit is contained in:
Nick Clifton
2014-06-26 09:12:55 +01:00
parent 4395285e33
commit 60d77146a2
2 changed files with 25 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
bfd/ChangeLog
View File

@ -1,3 +1,10 @@
2014-06-26 Nick Clifton <nickc@redhat.com>
PR binutils/16949
* dwarf2.c (is_str_attr): New function.
(find_abstract_instance_name): Use it to determine when an
attribute has a string value.
2014-06-24 Alan Modra <amodra@gmail.com> 2014-06-24 Alan Modra <amodra@gmail.com>
* elf32-ppc.c (ppc_elf_size_dynamic_sections): Arrange to keep * elf32-ppc.c (ppc_elf_size_dynamic_sections): Arrange to keep

22
bfd/dwarf2.c
View File

@ -911,6 +911,14 @@ read_abbrevs (bfd *abfd, bfd_uint64_t offset, struct dwarf2_debug *stash)
return abbrevs; return abbrevs;
} }
/* Returns true if the form is one which has a string value. */
static inline bfd_boolean
is_str_attr (enum dwarf_form form)
{
return form == DW_FORM_string || form == DW_FORM_strp || form == DW_FORM_GNU_strp_alt;
}
/* Read an attribute value described by an attribute form. */ /* Read an attribute value described by an attribute form. */
static bfd_byte * static bfd_byte *
@ -2201,7 +2209,7 @@ find_abstract_instance_name (struct comp_unit *unit,
case DW_AT_name: case DW_AT_name:
/* Prefer DW_AT_MIPS_linkage_name or DW_AT_linkage_name /* Prefer DW_AT_MIPS_linkage_name or DW_AT_linkage_name
over DW_AT_name. */ over DW_AT_name. */
if (name == NULL) if (name == NULL && is_str_attr (attr.form))
name = attr.u.str; name = attr.u.str;
break; break;
case DW_AT_specification: case DW_AT_specification:
@ -2209,7 +2217,10 @@ find_abstract_instance_name (struct comp_unit *unit,
break; break;
case DW_AT_linkage_name: case DW_AT_linkage_name:
case DW_AT_MIPS_linkage_name: case DW_AT_MIPS_linkage_name:
name = attr.u.str; /* PR 16949: Corrupt debug info can place
non-string forms into these attributes. */
if (is_str_attr (attr.name))
name = attr.u.str;
break; break;
default: default:
break; break;
@ -2381,13 +2392,16 @@ scan_unit_for_symbols (struct comp_unit *unit)
case DW_AT_name: case DW_AT_name:
/* Prefer DW_AT_MIPS_linkage_name or DW_AT_linkage_name /* Prefer DW_AT_MIPS_linkage_name or DW_AT_linkage_name
over DW_AT_name. */ over DW_AT_name. */
if (func->name == NULL) if (func->name == NULL && is_str_attr (attr.form))
func->name = attr.u.str; func->name = attr.u.str;
break; break;
case DW_AT_linkage_name: case DW_AT_linkage_name:
case DW_AT_MIPS_linkage_name: case DW_AT_MIPS_linkage_name:
func->name = attr.u.str; /* PR 16949: Corrupt debug info can place
non-string forms into these attributes. */
if (is_str_attr (attr.form))
func->name = attr.u.str;
break; break;
case DW_AT_low_pc: case DW_AT_low_pc: