diff options
author | Sandra Loosemore <sandra@codesourcery.com> | 2019-02-15 09:56:41 +0000 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2019-02-15 09:56:41 +0000 |
commit | 0f8b09b22234460cb5b8766a25066cf6b5f06842 (patch) | |
tree | 2bb4b8e5422dc39a98281abe7017003f5335a69c /gdbstub.c | |
parent | d8efe78e8039511b95c23d75bb48eca6873fbb0f (diff) | |
download | qemu-0f8b09b22234460cb5b8766a25066cf6b5f06842.zip qemu-0f8b09b22234460cb5b8766a25066cf6b5f06842.tar.gz qemu-0f8b09b22234460cb5b8766a25066cf6b5f06842.tar.bz2 |
gdbstub: Send a reply to the vKill packet.
Per the GDB remote protocol documentation
https://sourceware.org/gdb/current/onlinedocs/gdb/Packets.html#index-vKill-packet
the debug stub is expected to send a reply to the 'vKill' packet. At
least some versions of GDB crash if the gdb stub simply exits without
sending a reply. This patch fixes QEMU's gdb stub to conform to the
expected behavior.
Note that QEMU's existing handling of the legacy 'k' packet is
correct: in that case GDB does not expect a reply, and QEMU does not
send one.
Signed-off-by: Sandra Loosemore <sandra@codesourcery.com>
Message-id: 1550008033-26540-1-git-send-email-sandra@codesourcery.com
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Diffstat (limited to 'gdbstub.c')
-rw-r--r-- | gdbstub.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -1361,6 +1361,7 @@ static int gdb_handle_packet(GDBState *s, const char *line_buf) break; } else if (strncmp(p, "Kill;", 5) == 0) { /* Kill the target */ + put_packet(s, "OK"); error_report("QEMU: Terminated via GDBstub"); exit(0); } else { |