From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22295 invoked by alias); 8 Nov 2016 10:37:36 -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 22263 invoked by uid 89); 8 Nov 2016 10:37:35 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-4.8 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy=lionel, Lionel, stamp, contribute X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 08 Nov 2016 10:37:34 +0000 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 35535C057FAD; Tue, 8 Nov 2016 10:37:33 +0000 (UTC) Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id uA8AbVkm022764; Tue, 8 Nov 2016 05:37:32 -0500 From: Pedro Alves Subject: Re: Check for truncated registers in process_g_packet To: Lionel Flandrin , Simon Marchi References: <20161018111023.4hzeyfzzpaneyfds@localhost.localdomain> <33a1f569-995b-342a-dbb9-ea14ab377d1a@ericsson.com> <20161018160657.rdvxgcam3uibsgst@localhost.localdomain> Cc: gdb-patches@sourceware.org Message-ID: <6f2568ee-7677-63e8-2d51-65ac531b3a84@redhat.com> Date: Tue, 08 Nov 2016 10:37:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20161018160657.rdvxgcam3uibsgst@localhost.localdomain> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-SW-Source: 2016-11/txt/msg00153.txt.bz2 On 10/18/2016 05:07 PM, Lionel Flandrin wrote: >> This patch looks good to me, a few minor comments below about formatting. >> Someone else with the approval stamp must look at it, but hopefully it will >> save them a bit of work. > > Thank you for the feedback, here's the updated patch: This is OK. I've pushed it in. Do you have a copyright assignment in place with the FSF? I looked at the FSF records and couldn't find one. This patch is about the largest we can accept without one. If you'd like to contribute further, we'll need to get that sorted out. Feel free to follow up off list. Thanks for your contribution! -- Pedro Alves