Re: [libvirt] [PATCH 2/2] conf: Don't leak 'ram' video property on error

2013-02-21 Thread Eric Blake
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 | 1 + > 1 file changed, 1 insertion(+) ACK. Again, going with a more idiomatic cleanup label that cleans up on both s

[libvirt] [PATCH 2/2] conf: Don't leak 'ram' video property on error

2013-02-21 Thread Christophe Fergeau
It's only freed on normal returns from virDomainVideoDefParseXML, but not when erroring out. --- src/conf/domain_conf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 258ee93..0c75838 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/doma