qemumonitorjsontest: use virCPUDefNew()

virCPUDefPtr uses refcounting internally and must be allocated
using virCPUDefNew, otherwise virCPUDefFree would be a no-op.

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Fixes: fa2404bf4f91813431beb797fc30a1237a743823
Fixes: eee09435eec46553aac4fdf1c2d8f3214167bded
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Ján Tomko 2020-02-22 01:06:51 +01:00
parent 01e68455b0
commit 9aa2426795

View File

@ -2993,8 +2993,8 @@ testQemuMonitorJSONqemuMonitorJSONGetCPUModelComparison(const void *opaque)
"{\"return\":{\"result\":\"test\"}}") < 0) "{\"return\":{\"result\":\"test\"}}") < 0)
return -1; return -1;
if (VIR_ALLOC(cpu_a) < 0 || VIR_ALLOC(cpu_b) < 0) cpu_a = virCPUDefNew();
goto cleanup; cpu_b = virCPUDefNew();
cpu_a->model = g_strdup("cpu_a"); cpu_a->model = g_strdup("cpu_a");
cpu_b->model = g_strdup("cpu_b"); cpu_b->model = g_strdup("cpu_b");
@ -3045,8 +3045,8 @@ testQemuMonitorJSONqemuMonitorJSONGetCPUModelBaseline(const void *opaque)
"}") < 0) "}") < 0)
return -1; return -1;
if (VIR_ALLOC(cpu_a) < 0 || VIR_ALLOC(cpu_b) < 0) cpu_a = virCPUDefNew();
goto cleanup; cpu_b = virCPUDefNew();
cpu_a->model = g_strdup("cpu_a"); cpu_a->model = g_strdup("cpu_a");
cpu_b->model = g_strdup("cpu_b"); cpu_b->model = g_strdup("cpu_b");