Fix memory leak in virsh snapshot-list.

We were forgetting to release the memory allocated by
virDomainSnapshotListNames.  Free the memory properly.

Signed-off-by: Chris Lalancette <clalance@redhat.com>
This commit is contained in:
Chris Lalancette 2010-04-23 11:59:39 -04:00
parent 21c6cfc213
commit 52ae076a94

View File

@ -8360,7 +8360,7 @@ cmdSnapshotList(vshControl *ctl, const vshCmd *cmd)
int ret = FALSE; int ret = FALSE;
int numsnaps; int numsnaps;
char **names = NULL; char **names = NULL;
int actual; int actual = 0;
int i; int i;
xmlDocPtr xml = NULL; xmlDocPtr xml = NULL;
xmlXPathContextPtr ctxt = NULL; xmlXPathContextPtr ctxt = NULL;
@ -8447,6 +8447,8 @@ cleanup:
if (xml) if (xml)
xmlFreeDoc(xml); xmlFreeDoc(xml);
VIR_FREE(doc); VIR_FREE(doc);
for (i = 0; i < actual; i++)
VIR_FREE(names[i]);
VIR_FREE(names); VIR_FREE(names);
if (dom) if (dom)
virDomainFree(dom); virDomainFree(dom);