From e1d0471e587567647aae3ec7d81e12e88badd809 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Fri, 22 Aug 2014 10:30:44 -0400 Subject: [PATCH] commandtest: Resolve Coverity RESOURCE_LEAK Since '62f263a73' - Coverity complains if the !pidfile path is taken, then newfd1 would be leaked. --- tests/commandtest.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/commandtest.c b/tests/commandtest.c index b3287faa6f..5f52a0085d 100644 --- a/tests/commandtest.c +++ b/tests/commandtest.c @@ -1081,6 +1081,7 @@ static int test24(const void *unused ATTRIBUTE_UNUSED) unlink(pidfile); VIR_FREE(pidfile); virCommandFree(cmd); + VIR_FORCE_CLOSE(newfd1); /* coverity[double_close] */ VIR_FORCE_CLOSE(newfd2); VIR_FORCE_CLOSE(newfd3);