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

Re: Fix null pointer dereference in process_debug_info()

Browse Source 
read_bases has a potential null-pointer deref too, and without a
debug_info_p there isn't any point in calling read_bases.

	* dwarf.c (process_debug_info): Don't call read_bases when
	debug_info_p is NULL.
This commit is contained in:
Alan Modra
2024-04-12 08:55:02 +09:30
parent fcf8f3237c
commit 0f8adbf77d
1 changed files with 8 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
binutils/dwarf.c
View File

@@ -4236,7 +4236,7 @@ process_debug_info (struct dwarf_section * section,
Only needed for the top DIE on DWARFv5+.
No simiar treatment for loclists_base because there should
be no loclist attributes in top DIE. */
if (compunit.cu_version >= 5 && level == 0)
if (debug_info_p && compunit.cu_version >= 5 && level == 0)
{
int64_t stemp;
@@ -4248,17 +4248,14 @@ process_debug_info (struct dwarf_section * section,
compunit.cu_version,
debug_info_p);
if (debug_info_p)
/* This check was in place before, keep it. */
stemp = debug_info_p->rnglists_base;
if (stemp < 0)
{
/* This check was in place before, keep it. */
stemp = debug_info_p->rnglists_base;
if (stemp < 0)
{
warn (_("CU @ %#" PRIx64 " has has a negative rnglists_base "
"value of %#" PRIx64 " - treating as zero"),
debug_info_p->cu_offset, stemp);
debug_info_p->rnglists_base = 0;
}
warn (_("CU @ %#" PRIx64 " has has a negative rnglists_base "
"value of %#" PRIx64 " - treating as zero"),
debug_info_p->cu_offset, stemp);
debug_info_p->rnglists_base = 0;
}
}