Skip to content

Commit

Permalink
gdbstub: Add a missing case of signal number translation in gdbstub
Browse files Browse the repository at this point in the history
While using qemu with gdb "target remote" to debug an application that uses
fork and exec, the qemu process receives SIGSTOP every time the forked process
terminates (sending SIGCHLD).

This is caused by a missing call to gdb_signal_to_target in gdbstub.c, which
is fixed by this patch:

Signed-off-by: Martin Simmons <martin@lispworks.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
  • Loading branch information
m-lw authored and Michael Tokarev committed Nov 11, 2014
1 parent 5b009e4 commit f17b069
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion gdbstub.c
Expand Up @@ -823,7 +823,10 @@ static int gdb_handle_packet(GDBState *s, const char *line_buf)
action = *p++;
signal = 0;
if (action == 'C' || action == 'S') {
signal = strtoul(p, (char **)&p, 16);
signal = gdb_signal_to_target(strtoul(p, (char **)&p, 16));
if (signal == -1) {
signal = 0;
}
} else if (action != 'c' && action != 's') {
res = 0;
break;
Expand Down

0 comments on commit f17b069

Please sign in to comment.