From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 86895 invoked by alias); 5 Dec 2019 22:30:45 -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 84902 invoked by uid 89); 5 Dec 2019 22:30:31 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-10.8 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.1 spammy=HX-Languages-Length:418 X-HELO: mailsec106.isp.belgacom.be Received: from mailsec106.isp.belgacom.be (HELO mailsec106.isp.belgacom.be) (195.238.20.102) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 05 Dec 2019 22:30:28 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=skynet.be; i=@skynet.be; q=dns/txt; s=securemail; t=1575585029; x=1607121029; h=message-id:subject:from:to:date:in-reply-to:references: mime-version:content-transfer-encoding; bh=qJmc+IhFlp2sYJsQKGb0tHqKYGUdzexxcxqyjF4FHwU=; b=lxgkIKVOwU+Is6aY34Q3liRMlSWa5ZexUV5uUYDkPubIkjkCLfulMFap cU3vsYzizZ1BlONZNQVtgaqMhfBDwA==; IronPort-SDR: mABbiUWfNEuRAAELWpjoNAMdT7Ro2sUxnnfrh3VWtd3jJLOosXBEW7knKrucqlCXL3f1587tVu LOOvu3WLFl5vpaZDSWIN9HzWvnG7qty881LogyyZo1HX7ZQY0+zjVt/SHOsjSUg200duKyU8CS X/sfIQ+S0gxVbNGo4or2f9tmc02xZrg6q+rZI5Iq0YRpNGJayLECmrUTa3PYEF33R0W5Pzq+MH A6alloPqUx5GD11Pj92fBreYXgsVrZGlVPA6l/GwptJJ7JkokvDVvIbvgNQh7y9FNz06D+6DvM LgA= Received: from 136.173-134-109.adsl-dyn.isp.belgacom.be (HELO md) ([109.134.173.136]) by relay.skynet.be with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 05 Dec 2019 23:30:26 +0100 Message-ID: Subject: Re: [RFA] Fix leaks when pruning inferiors. From: Philippe Waroquiers To: Pedro Alves , gdb-patches@sourceware.org Date: Thu, 05 Dec 2019 22:30:00 -0000 In-Reply-To: References: <20191201155220.15971-1-philippe.waroquiers@skynet.be> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.30.5-1.1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2019-12/txt/msg00211.txt.bz2 On Thu, 2019-12-05 at 16:42 +0000, Pedro Alves wrote: > Please declare ss_next here instead of at the top. > You can omit the redundant "struct" while at it: > > inferior *next = ss->next; > delete_inferior (ss); > ss = next; > > OK with that change. Thanks! Pushed after fixing the above, and the ChangeLog message as suggested by Kevin. Thanks for the review. Philippe