From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dario Faggioli <dario.faggioli@citrix.com>
Subject: Re: [PATCH 12/28] libxl: only free cputopology if it
 was allocated in libxl__get_numa_candidate
Date: Wed, 18 Sep 2013 14:54:03 +0200
Message-ID: <1379508843.18543.113.camel@Abyss>
References: <1379475484-25993-1-git-send-email-mattjd@gmail.com>
	<1379475484-25993-13-git-send-email-mattjd@gmail.com>
	<1379491026.18543.41.camel@Abyss>
	<CA+nUz_+XuLy=tnWyfpcbQa3PZv3bsL-5s=EtNWsq9s6xOUgxGg@mail.gmail.com>
	<1379495450.18543.84.camel@Abyss>
	<CA+nUz_Jxq-vpGVbpqudouEC8+Dsmj0ns-48Mbvk0gE5XVp3+1w@mail.gmail.com>
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1717274165565814126=="
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <CA+nUz_Jxq-vpGVbpqudouEC8+Dsmj0ns-48Mbvk0gE5XVp3+1w@mail.gmail.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Matthew Daley <mattjd@gmail.com>
Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>, Ian Jackson <ian.jackson@eu.citrix.com>, Ian Campbell <ian.campbell@citrix.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

--===============1717274165565814126==
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="=-acxO9IhmvZLDC+2ANQJ2"

--=-acxO9IhmvZLDC+2ANQJ2
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On mer, 2013-09-18 at 21:30 +1200, Matthew Daley wrote:
> On Wed, Sep 18, 2013 at 9:10 PM, Dario Faggioli
>
> > Anyway, for libxl_cputopology_list_free() (and also for
> > libxl_numainfo_list_free() if I really decide to patch it to make it
> > more "libxl style"), especially considering that something similar to
> > this happens in other places (as we're seeing in 13/28), is there
> > anything we can do for preventing this to come up again in future scans=
?
>=20
> Simply marking the Defects in Coverity as "False Positive" (which I
> just did before) prevents them from bugging us in the future.
>=20
Oh, awesome!

> A quick grep shows that the functions are only called in ~11 spots
> AFAICT; that's still small enough to sensibly allow manual overriding
> if any of the other invocations are complained about.=20
>
Sounds good, thanks for explaining this. :-)

Regards,
Dario

--=20
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


--=-acxO9IhmvZLDC+2ANQJ2
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.14 (GNU/Linux)

iEYEABECAAYFAlI5omsACgkQk4XaBE3IOsTA3wCcCZUb3HErm0HNzVyl1E2QWka2
NgcAnA34OzG8ssaPMdKuQuqPrWR+4KG8
=Wzy8
-----END PGP SIGNATURE-----

--=-acxO9IhmvZLDC+2ANQJ2--


--===============1717274165565814126==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

--===============1717274165565814126==--