diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 30766c113e..0061b6f355 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +2020-01-31 Alan Modra + + PR 4110 + * elf.c (setup_group): Don't clear entire section contents, + just the padding after group flags. Release alloc'd memory + after a seek or read failure. + 2020-01-16 Jon Turney * peXXigen.c (pe_is_repro): New function. diff --git a/bfd/elf.c b/bfd/elf.c index a8d98a60f4..5e6b9a0f41 100644 --- a/bfd/elf.c +++ b/bfd/elf.c @@ -676,8 +676,6 @@ setup_group (bfd *abfd, Elf_Internal_Shdr *hdr, asection *newsect) continue; } - memset (shdr->contents, 0, amt); - if (bfd_seek (abfd, shdr->sh_offset, SEEK_SET) != 0 || (bfd_bread (shdr->contents, shdr->sh_size, abfd) != shdr->sh_size)) @@ -692,7 +690,8 @@ setup_group (bfd *abfd, Elf_Internal_Shdr *hdr, asection *newsect) /* PR 17510: If the group contents are even partially corrupt, do not allow any of the contents to be used. */ - memset (shdr->contents, 0, amt); + bfd_release (abfd, shdr->contents); + shdr->contents = NULL; continue; } @@ -712,6 +711,7 @@ setup_group (bfd *abfd, Elf_Internal_Shdr *hdr, asection *newsect) idx = H_GET_32 (abfd, src); if (src == shdr->contents) { + dest->shdr = NULL; dest->flags = idx; if (shdr->bfd_section != NULL && (idx & GRP_COMDAT)) shdr->bfd_section->flags