From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6471 invoked by alias); 19 Apr 2019 11:04:01 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 6462 invoked by uid 89); 19 Apr 2019 11:04:01 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-18.3 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3 autolearn=ham version=3.3.1 spammy=HX-Languages-Length:2610, terminate X-HELO: gnu.wildebeest.org Received: from wildebeest.demon.nl (HELO gnu.wildebeest.org) (212.238.236.112) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 19 Apr 2019 11:03:59 +0000 Received: from tarox.wildebeest.org (tarox.wildebeest.org [172.31.17.39]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id 4E0EB3085EC7; Fri, 19 Apr 2019 13:03:56 +0200 (CEST) Received: by tarox.wildebeest.org (Postfix, from userid 1000) id CAE8A40782EC; Fri, 19 Apr 2019 13:03:55 +0200 (CEST) Date: Fri, 19 Apr 2019 11:04:00 -0000 From: Mark Wielaard To: Philippe Waroquiers Cc: gdb-patches@sourceware.org, Pedro Alves , Sergio Durigan Junior Subject: Re: [RFA] Fix GDB 8.3 regression crash when registers cannot be modified. Message-ID: <20190419110355.GA24396@tarox.wildebeest.org> References: <20190413082817.29343-1-philippe.waroquiers@skynet.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190413082817.29343-1-philippe.waroquiers@skynet.be> User-Agent: Mutt/1.5.21 (2010-09-15) X-SW-Source: 2019-04/txt/msg00371.txt.bz2 Hi, On Sat, Apr 13, 2019 at 10:28:17AM +0200, Philippe Waroquiers wrote: > 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_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 > > struct infcall_suspend_state_deleter > 2019-04-13 Philippe Waroquiers > > * inferior.h (struct infcall_suspend_state_deleter): > Catch exception in destructor to avoid crash. What is the current status of this patch? We really need something like this to enable the valgrind gdbserver integration testing again on Fedora 30/rawhide. Thanks, Mark > --- > gdb/inferior.h | 14 +++++++++++++- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/gdb/inferior.h b/gdb/inferior.h > index 2d1bb97a28..4d84afac6a 100644 > --- a/gdb/inferior.h > +++ b/gdb/inferior.h > @@ -68,7 +68,19 @@ struct infcall_suspend_state_deleter > { > void operator() (struct infcall_suspend_state *state) const > { > - restore_infcall_suspend_state (state); > + TRY > + { > + restore_infcall_suspend_state (state); > + } > + CATCH (e, RETURN_MASK_ALL) > + { > + /* If we are restoring the inferior state due to an exception, > + some error message will be printed. So, only warn the user > + when we cannot restore during normal execution. */ > + if (!std::uncaught_exception ()) > + warning (_("Failed to restore inferior state: %s"), e.message); > + } > + END_CATCH > } > }; > > -- > 2.20.1