From ba9c7a3026d11dfb3afdc393da754ec3a4804be2 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Mon, 4 Feb 2013 09:57:00 -0500 Subject: [PATCH] netdev_vlan_conf: Resolve memory leak found by Valgrind. The 'trunk' is filled in with virXPathString() value, but was never VIR_FREE()'d. --- src/conf/netdev_vlan_conf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/conf/netdev_vlan_conf.c b/src/conf/netdev_vlan_conf.c index 9207184fc1..13ba8c6ab8 100644 --- a/src/conf/netdev_vlan_conf.c +++ b/src/conf/netdev_vlan_conf.c @@ -32,7 +32,7 @@ virNetDevVlanParse(xmlNodePtr node, xmlXPathContextPtr ctxt, virNetDevVlanPtr de { int ret = -1; xmlNodePtr save = ctxt->node; - const char *trunk; + const char *trunk = NULL; xmlNodePtr *tagNodes = NULL; int nTags, ii; @@ -103,6 +103,7 @@ virNetDevVlanParse(xmlNodePtr node, xmlXPathContextPtr ctxt, virNetDevVlanPtr de error: ctxt->node = save; VIR_FREE(tagNodes); + VIR_FREE(trunk); if (ret < 0) virNetDevVlanClear(def); return ret; -- 2.47.2