diff options
author | Doug Gilmore <Doug.Gilmore@imgtec.com> | 2017-06-28 02:54:22 +0100 |
---|---|---|
committer | Maciej W. Rozycki <macro@imgtec.com> | 2017-06-28 02:54:22 +0100 |
commit | 41664b45ab6387184bd0e7512fcf7a72c58744e7 (patch) | |
tree | c2c3df35f53f8a8d9ec76db7e96d066c0ef195f1 /gas/doc/c-arm.texi | |
parent | 819e1f86976dbbd13e0de004cdd3e3089e4c3fc0 (diff) | |
download | gdb-41664b45ab6387184bd0e7512fcf7a72c58744e7.zip gdb-41664b45ab6387184bd0e7512fcf7a72c58744e7.tar.gz gdb-41664b45ab6387184bd0e7512fcf7a72c58744e7.tar.bz2 |
Fix PR 21337: segfault when re-reading symbols.
Fix issue exposed by commit 3e29f34.
The basic issue is that section data referenced through an objfile
pointer can also be referenced via the program-space data pointer,
although via a separate mapping mechanism, which is set up by
update_section_map. Thus once section data attached to an objfile
pointer is released, the section map associated with the program-space
data pointer must be marked dirty to ensure that update_section_map is
called to prevent stale data being referenced. For the matter at hand
this marking is being done via a call to objfiles_changed.
Before commit 3e29f34 objfiles_changed could be called after all of
the objfile pointers were processed in reread_symbols since section
data references via the program-space data pointer would not occur in
the calls of read_symbols performed by reread_symbols.
With commit 3e29f34 MIPS target specific calls to find_pc_section were
added to the code for DWARF information processing, which is called
via read_symbols. Thus in reread_symbols the call to objfiles_changed
needs to be called before calling read_symbols, otherwise stale
section data can be referenced.
Thanks to Luis Machado for providing text for the main comment
associated with the change.
gdb/
2017-06-28 Doug Gilmore <Doug.Gilmore@imgtec.com>
PR gdb/21337
* symfile.c (reread_symbols): Call objfiles_changed just before
read_symbols.
gdb/testsuite/
2017-06-28 Doug Gilmore <Doug.Gilmore@imgtec.com>
PR gdb/21337
* gdb.base/reread-readsym.exp: New file.
* gdb.base/reread-readsym.c: New file.
Diffstat (limited to 'gas/doc/c-arm.texi')
0 files changed, 0 insertions, 0 deletions