On Wed, Oct 11, 2017 at 05:03:05PM -0400, John Ferlan wrote:
Need to free @groups in the parent on success similar to other
APIs (virFile*) which use virGetGroupList and virFork.
Reported by Coverity.
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
src/util/vircommand.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/util/vircommand.c b/src/util/vircommand.c
index 41a61da49f..6cd76a560e 100644
--- a/src/util/vircommand.c
+++ b/src/util/vircommand.c
@@ -606,6 +606,7 @@ virExec(virCommandPtr cmd)
cmd->pid = pid;
+ VIR_FREE(groups);
VIR_FREE(binarystr);
return 0;
Reviewed-by: Erik Skultety <eskultet(a)redhat.com>