aboutsummaryrefslogtreecommitdiff
path: root/gdb/bsd-kvm.c
diff options
context:
space:
mode:
authorPhilippe Waroquiers <philippe.waroquiers@skynet.be>2019-04-13 10:22:41 +0200
committerPhilippe Waroquiers <philippe.waroquiers@skynet.be>2019-04-19 14:11:51 +0200
commitee3c5f8968e7b43854204898fa46933b3a1b5991 (patch)
tree59b78218902b3c266b13401735161727c92d74dc /gdb/bsd-kvm.c
parentd563b953142db796f61425a1a300f0feddcdcd55 (diff)
downloadgdb-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