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

Fix NULL pointer indirection when parsing corrupt DWARF data.

Browse Source 
PR 29290
	* dwarf.c (read_and_display_attr_value): Check that debug_info_p
	is set before dereferencing it.
This commit is contained in:
Nick Clifton
2022-06-27 13:30:35 +01:00
parent 2a4fc266db
commit e98e7d9a70
2 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
binutils/ChangeLog
View File

@ -1,7 +1,14 @@
2022-06-27 Nick Clifton <nickc@redhat.com>
PR 29290
* dwarf.c (read_and_display_attr_value): Check that debug_info_p
is set before dereferencing it.
2022-06-27 Nick Clifton <nickc@redhat.com>
* dwarf.c (fetch_indexed_string): Do not use length of first table
in string section as the length of every table in the section.
* testsuite/binutils-all/pr26112.r: Update expected output.
2022-06-22 Kumar N, Bhuvanendra <Kavitha.Natarajan@amd.com>

11
binutils/dwarf.c
View File

@ -2694,18 +2694,17 @@ read_and_display_attr_value (unsigned long attribute,
{
const char *suffix = section ? strrchr (section->name, '.') : NULL;
bool dwo = suffix && strcmp (suffix, ".dwo") == 0;
const char *strng;
strng = fetch_indexed_string (uvalue, this_set, offset_size, dwo,
debug_info_p ? debug_info_p->str_offsets_base : 0);
if (do_wide)
/* We have already displayed the form name. */
printf (_("%c(offset: 0x%s): %s"), delimiter,
dwarf_vmatoa ("x", uvalue),
fetch_indexed_string (uvalue, this_set, offset_size, dwo,
debug_info_p->str_offsets_base));
dwarf_vmatoa ("x", uvalue), strng);
else
printf (_("%c(indexed string: 0x%s): %s"), delimiter,
dwarf_vmatoa ("x", uvalue),
fetch_indexed_string (uvalue, this_set, offset_size, dwo,
debug_info_p->str_offsets_base));
dwarf_vmatoa ("x", uvalue), strng);
}
break;