diff options
| author | Alan Modra <amodra@gmail.com> | 2018-05-07 22:41:47 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2018-05-07 22:50:31 +0930 |
| commit | ce4ec1a9b6c442a9feefa18dd8734372a718665c (patch) | |
| tree | cebd17c4a753f5d271f2aa4fbb0a8637cb15feb5 /binutils/objcopy.c | |
| parent | 7bd8862c3ad0ee291d27837ae3cd30288a00b922 (diff) | |
| download | gdb-ce4ec1a9b6c442a9feefa18dd8734372a718665c.zip gdb-ce4ec1a9b6c442a9feefa18dd8734372a718665c.tar.gz gdb-ce4ec1a9b6c442a9feefa18dd8734372a718665c.tar.bz2 | |
Bug 23142, SIGSEGV in is_strip_section
PR 23142
* objcopy.c (group_signature): Don't accept groups that use a
symbol table other than the one we've read.
Diffstat (limited to 'binutils/objcopy.c')
| -rw-r--r-- | binutils/objcopy.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/binutils/objcopy.c b/binutils/objcopy.c index 61c513b..fadc957 100644 --- a/binutils/objcopy.c +++ b/binutils/objcopy.c @@ -1211,14 +1211,13 @@ group_signature (asection *group) return NULL; ghdr = &elf_section_data (group)->this_hdr; - if (ghdr->sh_link < elf_numsections (abfd)) + if (ghdr->sh_link == elf_onesymtab (abfd)) { const struct elf_backend_data *bed = get_elf_backend_data (abfd); - Elf_Internal_Shdr *symhdr = elf_elfsections (abfd) [ghdr->sh_link]; + Elf_Internal_Shdr *symhdr = &elf_symtab_hdr (abfd); - if (symhdr->sh_type == SHT_SYMTAB - && ghdr->sh_info > 0 - && ghdr->sh_info < (symhdr->sh_size / bed->s->sizeof_sym)) + if (ghdr->sh_info > 0 + && ghdr->sh_info < symhdr->sh_size / bed->s->sizeof_sym) return isympp[ghdr->sh_info - 1]; } return NULL; |