diff options
author | Tom de Vries <tdevries@suse.de> | 2021-11-29 16:19:16 +0100 |
---|---|---|
committer | Tom de Vries <tdevries@suse.de> | 2021-11-29 16:19:16 +0100 |
commit | 49fa1332a69bc4b09d2cc7db587e27ea30f2a29d (patch) | |
tree | 1bf3c62ec0c6c9a9220221906aeb24c1bc322ce7 /binutils | |
parent | 8fee99c3c8e25f2708bddde345c192dd2f3e7e08 (diff) | |
download | gdb-49fa1332a69bc4b09d2cc7db587e27ea30f2a29d.zip gdb-49fa1332a69bc4b09d2cc7db587e27ea30f2a29d.tar.gz gdb-49fa1332a69bc4b09d2cc7db587e27ea30f2a29d.tar.bz2 |
[gdb/symtab] Fix segfault in search_one_symtab
PR28539 describes a segfault in lambda function search_one_symtab due to
psymbol_functions::expand_symtabs_matching calling expansion_notify with a
nullptr symtab:
...
struct compunit_symtab *symtab =
psymtab_to_symtab (objfile, ps);
if (expansion_notify != NULL)
if (!expansion_notify (symtab))
return false;
...
This happens as follows. The partial symtab ps is a dwarf2_include_psymtab
for some header file:
...
(gdb) p ps.filename
$5 = 0x64fcf80 "/usr/include/c++/11/bits/stl_construct.h"
...
The includer of ps is a shared symtab for a partial unit, with as user:
...
(gdb) p ps.includer().user.filename
$11 = 0x64fc9f0 \
"/usr/src/debug/llvm13-13.0.0-1.2.x86_64/tools/clang/lib/AST/Decl.cpp"
...
The call to psymtab_to_symtab expands the Decl.cpp symtab (and consequently
the shared symtab), but returns nullptr because:
...
struct dwarf2_include_psymtab : public partial_symtab
{
...
compunit_symtab *get_compunit_symtab (struct objfile *objfile) const override
{
return nullptr;
}
...
Fix this by returning the Decl.cpp symtab instead, which fixes the segfault
in the PR.
Tested on x86_64-linux.
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=28539
Diffstat (limited to 'binutils')
0 files changed, 0 insertions, 0 deletions