Fix seg-fault in the linker when examining a corrupt binary.
PR ld/20909 * aoutx.h (aout_link_add_symbols): Fix off-by-one error in check for an illegal string offset.
This commit is contained in:
parent
0c1b455e29
commit
531336e3a0
|
@ -1,3 +1,9 @@
|
|||
2016-12-02 Nick Clifton <nickc@redhat.com>
|
||||
|
||||
PR ld/20909
|
||||
* aoutx.h (aout_link_add_symbols): Fix off-by-one error in check
|
||||
for an illegal string offset.
|
||||
|
||||
2016-12-02 Gary Benson <gbenson@redhat.com>
|
||||
|
||||
* elf.c (_bfd_elf_make_section_from_shdr): Pass offset to
|
||||
|
|
|
@ -3031,10 +3031,9 @@ aout_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
|
|||
continue;
|
||||
|
||||
/* PR 19629: Corrupt binaries can contain illegal string offsets. */
|
||||
if (GET_WORD (abfd, p->e_strx) > obj_aout_external_string_size (abfd))
|
||||
if (GET_WORD (abfd, p->e_strx) >= obj_aout_external_string_size (abfd))
|
||||
return FALSE;
|
||||
name = strings + GET_WORD (abfd, p->e_strx);
|
||||
|
||||
value = GET_WORD (abfd, p->e_value);
|
||||
flags = BSF_GLOBAL;
|
||||
string = NULL;
|
||||
|
|
Loading…
Reference in New Issue