diff options
author | Philippe Waroquiers <philippe.waroquiers@skynet.be> | 2019-04-13 10:22:41 +0200 |
---|---|---|
committer | Philippe Waroquiers <philippe.waroquiers@skynet.be> | 2019-04-19 14:11:51 +0200 |
commit | ee3c5f8968e7b43854204898fa46933b3a1b5991 (patch) | |
tree | 59b78218902b3c266b13401735161727c92d74dc /gdb/bsd-kvm.c | |
parent | d563b953142db796f61425a1a300f0feddcdcd55 (diff) | |
download | gdb-ee3c5f8968e7b43854204898fa46933b3a1b5991.zip gdb-ee3c5f8968e7b43854204898fa46933b3a1b5991.tar.gz gdb-ee3c5f8968e7b43854204898fa46933b3a1b5991.tar.bz2 |
Fix GDB crash when registers cannot be modified.
This crash was detected when using GDB with the valgrind gdbserver.
To reproduce:
valgrind sleep 10000
In another window:
gdb
target remote | vgdb
p printf("make sleep print something\n")
=>
terminate called after throwing an instance of 'gdb_exception_error'
Aborted
The problem is that the valgrind gdbserver does not allow to change
registers when the inferior is blocked in a system call.
GDB then raises an exception. The exception causes the destructor
of
typedef std::unique_ptr<infcall_suspend_state, infcall_suspend_state_deleter>
infcall_suspend_state_up;
to be called. This destructor itself tries to restore the value of
the registers, and fails similarly. We must catch the exception in
the destructor to avoid crashing GDB.
If the destructor encounters a problem, no warning is produced if
there is an uncaught exception, as in this case, the user will already
be informed of a problem via this exception.
With this change, no crash anymore, and all the valgrind 3.15 tests
pass succesfully.
gdb/ChangeLog
2019-04-19 Philippe Waroquiers <philippe.waroquiers@skynet.be>
* inferior.h (struct infcall_suspend_state_deleter):
Catch exception in destructor to avoid crash.
Diffstat (limited to 'gdb/bsd-kvm.c')
0 files changed, 0 insertions, 0 deletions