[libvirt] [PATCH 1/2] conf: Don't leak 'primary' video property on error
Christophe Fergeau
cfergeau at redhat.com
Fri Feb 22 08:40:32 UTC 2013
On Thu, Feb 21, 2013 at 01:07:27PM -0700, Eric Blake wrote:
> On 02/21/2013 08:49 AM, Christophe Fergeau wrote:
> > It's only freed on normal returns from virDomainVideoDefParseXML,
> > but not when erroring out.
> > ---
> > src/conf/domain_conf.c | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
>
> ACK; your fix is reasonable. A more idiomatic fix might be to coalesce
> the error and normal exits into a single cleanup: label; in fact,
> Peter's 16-part series on improving domain_conf.c may have done just
> that. But as I haven't gotten that far in reviewing his series, and
> this is a real fix, you can go ahead and push and Peter can deal with
> any rebase fallout.
Thanks, I've pushed the 2 patches now.
Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130222/18a91101/attachment-0001.sig>
More information about the libvir-list
mailing list