yggdrasil/binutils-gdb
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix an attempt to allocate an unreasonably large amount of memory when parsing a corrupt ELF file.

Browse Source 
PR  29924
	* objdump.c (load_specific_debug_section): Check for excessively
	large sections.
This commit is contained in:
Nick Clifton 2022-12-21 11:51:23 +00:00
parent a7a32d588f
commit 75393a2d54
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
binutils/ChangeLog
View File

@ -1,3 +1,9 @@
2022-12-21 Nick Clifton <nickc@redhat.com>
PR 29924
* objdump.c (load_specific_debug_section): Check for excessively
large sections.
2022-12-19 Tom Tromey <tromey@adacore.com>
* dwarf-mode.el (dwarf-do-refresh): Avoid compiler warning.

4
binutils/objdump.c
View File

@ -4225,7 +4225,9 @@ load_specific_debug_section (enum dwarf_section_display_enum debug,
section->size = bfd_section_size (sec);
/* PR 24360: On 32-bit hosts sizeof (size_t) < sizeof (bfd_size_type). */
alloced = amt = section->size + 1;
if (alloced != amt || alloced == 0)
if (alloced != amt
|| alloced == 0
|| (bfd_get_size (abfd) != 0 && alloced >= bfd_get_size (abfd)))
{
section->start = NULL;
free_debug_section (debug);