[libvirt] [PATCH 1/3] commandtest: Resolve Coverity RESOURCE_LEAK
Michal Privoznik
mprivozn at redhat.com
Fri Aug 22 15:49:01 UTC 2014
On 22.08.2014 17:28, John Ferlan wrote:
> Since '62f263a73' - Coverity complains if the !pidfile path is taken,
> then newfd1 would be leaked.
>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> tests/commandtest.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tests/commandtest.c b/tests/commandtest.c
> index b3287fa..5f52a00 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);
>
ACK
Michal
More information about the libvir-list
mailing list