Fix GDB 8.3 regression crash when registers cannot be modified.
authorPhilippe Waroquiers <philippe.waroquiers@skynet.be>
Sat, 13 Apr 2019 07:13:53 +0000 (09:13 +0200)
committerPhilippe Waroquiers <philippe.waroquiers@skynet.be>
Fri, 19 Apr 2019 11:51:54 +0000 (13:51 +0200)
commitbf9a7cea9c88de232f07151614562f81b52630a0
tree9c47b5b4cdac87ba18e9e834805cbe6f56634f00
parent6530876a88413b42b1b6b0ccbfe850fd590df0e9
Fix GDB 8.3 regression 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_RETURN_MASK_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.

Note: when this patch is approved, I will push an equivalent patch
on master, but with TRY/CATCH/e.message () replaced by
try/catch/e.what ().

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.
gdb/ChangeLog
gdb/inferior.h