diff options
author | Doug Evans <dje@google.com> | 2017-09-28 09:20:59 -0700 |
---|---|---|
committer | Doug Evans <dje@google.com> | 2017-09-28 09:24:48 -0700 |
commit | 08302ed2cb8b175e3bbd4308879154baf64e6d98 (patch) | |
tree | fd30f1e1139151b6923393fa2c47ca2f2c072fce /gdb/dwarf2read.c | |
parent | 231ca753239211b72eb761593083a0ee859274e7 (diff) |
(open_and_init_dwp_file): Protect against some segvs
This is a "tiny patch", no assignment required.
2017-09-28 Alexander Shaposhnikov <alexander.v.shaposhnikov@gmail.com>
* dwarf2read.c (open_and_init_dwp_file): Protect against dwp_file
having NULL cus or tus.
Diffstat (limited to 'gdb/dwarf2read.c')
-rw-r--r-- | gdb/dwarf2read.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index 7a59c1b435..1b15adced6 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -11199,7 +11199,8 @@ open_and_init_dwp_file (void) dwp_file->tus = create_dwp_hash_table (dwp_file, 1); /* The DWP file version is stored in the hash table. Oh well. */ - if (dwp_file->cus->version != dwp_file->tus->version) + if (dwp_file->cus && dwp_file->tus + && dwp_file->cus->version != dwp_file->tus->version) { /* Technically speaking, we should try to limp along, but this is pretty bizarre. We use pulongest here because that's the established @@ -11209,7 +11210,13 @@ open_and_init_dwp_file (void) pulongest (dwp_file->cus->version), pulongest (dwp_file->tus->version), dwp_name.c_str ()); } - dwp_file->version = dwp_file->cus->version; + + if (dwp_file->cus) + dwp_file->version = dwp_file->cus->version; + else if (dwp_file->tus) + dwp_file->version = dwp_file->tus->version; + else + dwp_file->version = 2; if (dwp_file->version == 2) bfd_map_over_sections (dwp_file->dbfd, dwarf2_locate_v2_dwp_sections, |