diff options
| author | Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> | 2023-08-08 10:13:58 +0200 |
|---|---|---|
| committer | Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> | 2023-08-08 10:13:58 +0200 |
| commit | 67be0489f28161239442dae38d4a14a38cb824de (patch) | |
| tree | 7484514ce53a86950ec1b845cda26b5700236579 /gdb/testsuite/gdb.python | |
| parent | aaa1d4b8f227c29f27a13a4a14622f0a3499e4b9 (diff) | |
| download | binutils-67be0489f28161239442dae38d4a14a38cb824de.zip binutils-67be0489f28161239442dae38d4a14a38cb824de.tar.gz binutils-67be0489f28161239442dae38d4a14a38cb824de.tar.bz2 | |
Guard against killing unrelated processes in amd64-disp-step.exp
When testing current gdb trunk on Solaris/amd64, the whole session was
reliably terminated by make check. I could trace this to the following
entry in gdb.arch/amd64-disp-step/gdb.log:
FAIL: gdb.arch/amd64-disp-step.exp: add into rcx: send_signal=on: get
inferior pid
Executing on target: kill -ALRM -1 (timeout = 300)
builtin_spawn -ignore SIGHUP kill -ALRM -1
If $inferior_pid doesn't refer to a single process for some reason, this
kill would terminate either a process group or the whole session.
This patch avoids this by ensuring that the pid arg is positive.
Tested on amd64-pc-solaris2.11 and x86_64-pc-linux-gnu.
Co-Authored-By: Andrew Burgess <aburgess@redhat.com>
Approved-By: Tom Tromey <tom@tromey.com>
Diffstat (limited to 'gdb/testsuite/gdb.python')
0 files changed, 0 insertions, 0 deletions