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

bfd: Avoid signed overflow for new_size adjustment

Browse Source 
When bfd_size_type is unsigned 64-bit integer and sizeof is unsigned
32-bit integer, subtraction in

*new_size += sizeof (Elf32_External_Chdr) - sizeof (Elf64_External_Chdr);

will overflow.  Use

*new_size -= sizeof (Elf64_External_Chdr) - sizeof (Elf32_External_Chdr);

to avoid overflow.

	PR binutils/29860
	* compress.c (bfd_convert_section_setup): Avoid signed overflow
	for new_size adjustment.
This commit is contained in:
H.J. Lu
2022-12-06 12:54:43 -08:00
parent 53fd08b60d
commit e25466730d
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
bfd/compress.c
View File

@ -306,7 +306,7 @@ bfd_convert_section_setup (bfd *ibfd, asection *isec, bfd *obfd,
if (hdr_size == sizeof (Elf32_External_Chdr)) if (hdr_size == sizeof (Elf32_External_Chdr))
*new_size += sizeof (Elf64_External_Chdr) - sizeof (Elf32_External_Chdr); *new_size += sizeof (Elf64_External_Chdr) - sizeof (Elf32_External_Chdr);
else else
*new_size += sizeof (Elf32_External_Chdr) - sizeof (Elf64_External_Chdr); *new_size -= sizeof (Elf64_External_Chdr) - sizeof (Elf32_External_Chdr);
return true; return true;
} }