diff options
| author | Thomas Hebb <tommyhebb@gmail.com> | 2022-04-29 21:17:58 -0700 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2022-04-30 19:21:11 +0930 |
| commit | 16089f320a9226e7cdb73e9fb4266d9e450085b2 (patch) | |
| tree | bb6e791e1b2862d6204e645bf27ca312423c23da /opcodes/riscv-dis.c | |
| parent | 2e920d702b43c6d21ebd1e8a49c9e976a0d2cde6 (diff) | |
| download | gdb-16089f320a9226e7cdb73e9fb4266d9e450085b2.zip gdb-16089f320a9226e7cdb73e9fb4266d9e450085b2.tar.gz gdb-16089f320a9226e7cdb73e9fb4266d9e450085b2.tar.bz2 | |
opcodes: don't assume ELF in riscv, csky, rl78, mep disassemblers
Currently, the get_disassembler() implementations for riscv, csky, and
rl78--and mep_print_insn() for mep--access ELF variants of union fields
without first checking that the bfd actually represents an ELF. This
causes undefined behavior and crashes when disassembling non-ELF files
(the "binary" BFD, for example). Fix that.
Diffstat (limited to 'opcodes/riscv-dis.c')
| -rw-r--r-- | opcodes/riscv-dis.c | 28 |
1 files changed, 12 insertions, 16 deletions
diff --git a/opcodes/riscv-dis.c b/opcodes/riscv-dis.c index bfaefa3..9ff3116 100644 --- a/opcodes/riscv-dis.c +++ b/opcodes/riscv-dis.c @@ -1002,24 +1002,20 @@ riscv_get_disassembler (bfd *abfd) { const char *default_arch = "rv64gc"; - if (abfd) + if (abfd && bfd_get_flavour (abfd) == bfd_target_elf_flavour) { - const struct elf_backend_data *ebd = get_elf_backend_data (abfd); - if (ebd) + const char *sec_name = get_elf_backend_data (abfd)->obj_attrs_section; + if (bfd_get_section_by_name (abfd, sec_name) != NULL) { - const char *sec_name = ebd->obj_attrs_section; - if (bfd_get_section_by_name (abfd, sec_name) != NULL) - { - obj_attribute *attr = elf_known_obj_attributes_proc (abfd); - unsigned int Tag_a = Tag_RISCV_priv_spec; - unsigned int Tag_b = Tag_RISCV_priv_spec_minor; - unsigned int Tag_c = Tag_RISCV_priv_spec_revision; - riscv_get_priv_spec_class_from_numbers (attr[Tag_a].i, - attr[Tag_b].i, - attr[Tag_c].i, - &default_priv_spec); - default_arch = attr[Tag_RISCV_arch].s; - } + obj_attribute *attr = elf_known_obj_attributes_proc (abfd); + unsigned int Tag_a = Tag_RISCV_priv_spec; + unsigned int Tag_b = Tag_RISCV_priv_spec_minor; + unsigned int Tag_c = Tag_RISCV_priv_spec_revision; + riscv_get_priv_spec_class_from_numbers (attr[Tag_a].i, + attr[Tag_b].i, + attr[Tag_c].i, + &default_priv_spec); + default_arch = attr[Tag_RISCV_arch].s; } } |