diff options
| author | Alan Modra <amodra@gmail.com> | 2022-01-01 13:52:26 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2022-01-01 14:22:09 +1030 |
| commit | aebb3d543da4d411d3d767912193c3849f59ff99 (patch) | |
| tree | 3386516764a12f078d71548b68e8080f3dcfc337 | |
| parent | bb67763979c2084af185a8bf6ead4a05a50e7b9d (diff) | |
| download | binutils-gdb-aebb3d543da4d411d3d767912193c3849f59ff99.tar.gz | |
ubsan: integer overflow in section filepos subtraction
* elf.c (assign_file_positions_for_non_load_sections): Avoid
signed integer overflow.
| -rw-r--r-- | bfd/elf.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/bfd/elf.c b/bfd/elf.c index 92c06f2e44f..d8a270d5219 100644 --- a/bfd/elf.c +++ b/bfd/elf.c @@ -6289,8 +6289,7 @@ assign_file_positions_for_non_load_sections (bfd *abfd, Elf_Internal_Shdr *hdr = &elf_section_data (sect)->this_hdr; if (hdr->sh_type != SHT_NOBITS) { - p->p_filesz = (sect->filepos - m->sections[0]->filepos - + hdr->sh_size); + p->p_filesz = sect->filepos - p->p_offset + hdr->sh_size; /* NB: p_memsz of the loadable PT_NOTE segment should be the same as p_filesz. */ if (p->p_type == PT_NOTE |