diff options
| author | Tom de Vries <tdevries@suse.de> | 2023-08-28 16:27:58 +0200 |
|---|---|---|
| committer | Tom de Vries <tdevries@suse.de> | 2023-08-28 16:27:58 +0200 |
| commit | 8d83f51b91072bc3d79061eba7e564ba9be6afd7 (patch) | |
| tree | c308385f7b86c05c4617e5051ae4924442297163 /gdb/dwarf2 | |
| parent | daafebb58dac3de93ac4696dd334530b762ed67f (diff) | |
| download | gdb-8d83f51b91072bc3d79061eba7e564ba9be6afd7.zip gdb-8d83f51b91072bc3d79061eba7e564ba9be6afd7.tar.gz gdb-8d83f51b91072bc3d79061eba7e564ba9be6afd7.tar.bz2 | |
[gdb/symtab] Handle self-reference in inherit_abstract_dies
Building gdb with gcc 7.5.0 and -flto -O2 -flto-partition=one generates a
self-referencing DIE:
...
<2><91dace>: Abbrev Number: 405 (DW_TAG_label)
<91dad0> DW_AT_abstract_origin: <0x91dace>
...
When encountering the self-reference DIE in inherit_abstract_dies we loop
following the abstract origin, effectively hanging gdb.
Fix this by handling self-referencing DIEs in the loop in
inherit_abstract_dies.
Tested on x86_64-linux.
Approved-By: Tom Tromey <tom@tromey.com>
PR symtab/30799
https://sourceware.org/bugzilla/show_bug.cgi?id=30799
Diffstat (limited to 'gdb/dwarf2')
| -rw-r--r-- | gdb/dwarf2/read.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c index eb4cb9b..5b32089 100644 --- a/gdb/dwarf2/read.c +++ b/gdb/dwarf2/read.c @@ -9935,8 +9935,15 @@ inherit_abstract_dies (struct die_info *die, struct dwarf2_cu *cu) if (attr == nullptr) break; + die_info *prev_child_origin_die = child_origin_die; child_origin_die = follow_die_ref (child_origin_die, attr, &child_origin_cu); + + if (prev_child_origin_die == child_origin_die) + { + /* Handle DIE with self-reference. */ + break; + } } /* If missing DW_AT_abstract_origin, try the corresponding child |