tools: Fix memory leak

Coverity found that commit id 'c661b675f' needed to create a cleanup
path to handle the closing of 'fp' if the virBitmapNewQuiet failed.
This commit is contained in:
John Ferlan 2016-04-14 09:55:21 -04:00
parent 6c09c17e0d
commit cb31d618fd

View File

@ -194,13 +194,13 @@ int virHostValidateNamespace(const char *hvname,
virBitmapPtr virHostValidateGetCPUFlags(void)
{
FILE *fp;
virBitmapPtr flags;
virBitmapPtr flags = NULL;
if (!(fp = fopen("/proc/cpuinfo", "r")))
return NULL;
if (!(flags = virBitmapNewQuiet(VIR_HOST_VALIDATE_CPU_FLAG_LAST)))
return NULL;
goto cleanup;
do {
char line[1024];
@ -246,6 +246,7 @@ virBitmapPtr virHostValidateGetCPUFlags(void)
virStringFreeListCount(tokens, ntokens);
} while (1);
cleanup:
VIR_FORCE_FCLOSE(fp);
return flags;