tests: cputest: use g_auto for virCPUData

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Ján Tomko 2021-08-20 15:57:56 +02:00
parent a32c2b2360
commit 12185e5a3a

View File

@ -404,7 +404,7 @@ cpuTestHasFeature(const void *arg)
const struct data *data = arg; const struct data *data = arg;
int ret = -1; int ret = -1;
virCPUDef *host = NULL; virCPUDef *host = NULL;
virCPUData *hostData = NULL; g_autoptr(virCPUData) hostData = NULL;
int result; int result;
if (!(host = cpuTestLoadXML(data->arch, data->host))) if (!(host = cpuTestLoadXML(data->arch, data->host)))
@ -434,7 +434,6 @@ cpuTestHasFeature(const void *arg)
ret = 0; ret = 0;
cleanup: cleanup:
virCPUDataFree(hostData);
virCPUDefFree(host); virCPUDefFree(host);
return ret; return ret;
} }
@ -552,7 +551,7 @@ cpuTestCPUID(bool guest, const void *arg)
{ {
const struct data *data = arg; const struct data *data = arg;
int ret = -1; int ret = -1;
virCPUData *hostData = NULL; g_autoptr(virCPUData) hostData = NULL;
char *hostFile = NULL; char *hostFile = NULL;
char *host = NULL; char *host = NULL;
virCPUDef *cpu = NULL; virCPUDef *cpu = NULL;
@ -596,7 +595,6 @@ cpuTestCPUID(bool guest, const void *arg)
cleanup: cleanup:
VIR_FREE(hostFile); VIR_FREE(hostFile);
VIR_FREE(host); VIR_FREE(host);
virCPUDataFree(hostData);
virCPUDefFree(cpu); virCPUDefFree(cpu);
VIR_FREE(result); VIR_FREE(result);
virObjectUnref(models); virObjectUnref(models);
@ -646,7 +644,7 @@ static int
cpuTestCPUIDSignature(const void *arg) cpuTestCPUIDSignature(const void *arg)
{ {
const struct data *data = arg; const struct data *data = arg;
virCPUData *hostData = NULL; g_autoptr(virCPUData) hostData = NULL;
char *hostFile = NULL; char *hostFile = NULL;
char *host = NULL; char *host = NULL;
int ret = -1; int ret = -1;
@ -661,7 +659,6 @@ cpuTestCPUIDSignature(const void *arg)
ret = cpuTestCompareSignature(data, hostData); ret = cpuTestCompareSignature(data, hostData);
cleanup: cleanup:
virCPUDataFree(hostData);
VIR_FREE(hostFile); VIR_FREE(hostFile);
VIR_FREE(host); VIR_FREE(host);
return ret; return ret;
@ -765,10 +762,10 @@ cpuTestUpdateLive(const void *arg)
virCPUDef *cpu = NULL; virCPUDef *cpu = NULL;
char *enabledFile = NULL; char *enabledFile = NULL;
char *enabled = NULL; char *enabled = NULL;
virCPUData *enabledData = NULL; g_autoptr(virCPUData) enabledData = NULL;
char *disabledFile = NULL; char *disabledFile = NULL;
char *disabled = NULL; char *disabled = NULL;
virCPUData *disabledData = NULL; g_autoptr(virCPUData) disabledData = NULL;
char *expectedFile = NULL; char *expectedFile = NULL;
virCPUDef *expected = NULL; virCPUDef *expected = NULL;
virDomainCapsCPUModels *hvModels = NULL; virDomainCapsCPUModels *hvModels = NULL;
@ -842,10 +839,8 @@ cpuTestUpdateLive(const void *arg)
virCPUDefFree(cpu); virCPUDefFree(cpu);
VIR_FREE(enabledFile); VIR_FREE(enabledFile);
VIR_FREE(enabled); VIR_FREE(enabled);
virCPUDataFree(enabledData);
VIR_FREE(disabledFile); VIR_FREE(disabledFile);
VIR_FREE(disabled); VIR_FREE(disabled);
virCPUDataFree(disabledData);
VIR_FREE(expectedFile); VIR_FREE(expectedFile);
virCPUDefFree(expected); virCPUDefFree(expected);
virObjectUnref(hvModels); virObjectUnref(hvModels);
@ -892,7 +887,7 @@ cpuTestJSONSignature(const void *arg)
{ {
const struct data *data = arg; const struct data *data = arg;
g_autoptr(virQEMUCaps) qemuCaps = NULL; g_autoptr(virQEMUCaps) qemuCaps = NULL;
virCPUData *hostData = NULL; g_autoptr(virCPUData) hostData = NULL;
qemuMonitorCPUModelInfo *modelInfo; qemuMonitorCPUModelInfo *modelInfo;
int ret = -1; int ret = -1;
@ -906,7 +901,6 @@ cpuTestJSONSignature(const void *arg)
ret = cpuTestCompareSignature(data, hostData); ret = cpuTestCompareSignature(data, hostData);
cleanup: cleanup:
virCPUDataFree(hostData);
return ret; return ret;
} }
#endif #endif