From 3906a8fc773a405b511faed030e4630ecbd4b8a9 Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Fri, 5 Aug 2016 19:54:11 +0100 Subject: Fix PR remote/20398: File-IO write always outputs "Quit" Commit bb7c96deb1a1 ("gdb/remote-fileio.c: Eliminate custom SIGINT signal handler") regressed the File-IO support. Failed output: (gdb) target remote :8888 Remote debugging using :8888 0x00008098 in _start () (gdb) c Continuing. Quit Quit Quit Quit Quit Quit Quit Quit Quit Quit Quit [Inferior 1 (Remote target) exited normally] Expected output: (gdb) target remote :8888 Remote debugging using :8888 0x00008098 in _start () (gdb) c Continuing. i: 0 i: 1 i: 2 i: 3 i: 4 i: 5 i: 6 i: 7 i: 8 i: 9 [Inferior 1 (Remote target) exited normally] The problem that the new File-IO quit handler forgets to check the quit flag before calling throwing a quit. gdb/ChangeLog: 2016-08-05 Pedro Alves PR remote/20398 * remote-fileio.c (remote_fileio_quit_handler): Check the quit flag before calling quit. --- gdb/remote-fileio.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'gdb/remote-fileio.c') diff --git a/gdb/remote-fileio.c b/gdb/remote-fileio.c index 93121aa..e35bd5b 100644 --- a/gdb/remote-fileio.c +++ b/gdb/remote-fileio.c @@ -308,7 +308,8 @@ static quit_handler_ftype *remote_fileio_o_quit_handler; static void remote_fileio_quit_handler (void) { - quit (); + if (check_quit_flag ()) + quit (); } static void -- cgit v1.1