From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38259) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bVafv-0008Ed-81 for qemu-devel@nongnu.org; Fri, 05 Aug 2016 04:40:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bVafp-00061d-6D for qemu-devel@nongnu.org; Fri, 05 Aug 2016 04:40:15 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40326) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bVafp-00061G-0F for qemu-devel@nongnu.org; Fri, 05 Aug 2016 04:40:09 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) (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 5CE5581256 for ; Fri, 5 Aug 2016 08:40:08 +0000 (UTC) Date: Fri, 5 Aug 2016 09:40:04 +0100 From: "Daniel P. Berrange" Message-ID: <20160805084004.GJ9933@redhat.com> Reply-To: "Daniel P. Berrange" References: <20160805082421.21994-1-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20160805082421.21994-1-marcandre.lureau@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH for-2.7 v4 00/36] Various memory leak fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: marcandre.lureau@redhat.com Cc: qemu-devel@nongnu.org, pbonzini@redhat.com, armbru@redhat.com On Fri, Aug 05, 2016 at 12:23:45PM +0400, marcandre.lureau@redhat.com wro= te: > From: Marc-Andr=C3=A9 Lureau >=20 > Hi, >=20 > Thanks to AddressSanitizer (ASAN), I found a number of direct leaks > worth fixing. Note that there are probably many indirect leaks left (I > am adding some here), I haven't investigated much yet. >=20 > There are still a number of direct leaks remaining, in particular in > the tests, but my libc doesn't give me good backtraces. >=20 > In order to easily switch to asan-enabled build, I make use of make > CFLAGS argument, which is why the first patch is also there. FWIW, you could probably use ./configure --extra-cflags=3D"....." which = is the currently supported way to inject extra custom flags into $CFLAGS. Regards, Daniel --=20 |: http://berrange.com -o- http://www.flickr.com/photos/dberrange= / :| |: http://libvirt.org -o- http://virt-manager.or= g :| |: http://autobuild.org -o- http://search.cpan.org/~danberr= / :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vn= c :|