External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-01-05 04:25:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

tests: qemuhotplug: Don't free the monitor object as part of @vm

Browse Source 
The test monitor should be freed separately so we need to remove the
pointer from the @vm object. This fixes a race condition crash in the
test introduced in commit a245abce43.
This commit is contained in:
Peter Krempa 2017-02-02 16:42:18 +01:00
parent 5ae7b0d8af
commit 5550dde150
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tests/qemuhotplugtest.c
View File

@ -365,6 +365,8 @@ struct testQemuHotplugCpuData {
static void static void
testQemuHotplugCpuDataFree(struct testQemuHotplugCpuData *data) testQemuHotplugCpuDataFree(struct testQemuHotplugCpuData *data)
{ {
qemuDomainObjPrivatePtr priv;
if (!data) if (!data)
return; return;
@ -375,7 +377,13 @@ testQemuHotplugCpuDataFree(struct testQemuHotplugCpuData *data)
VIR_FREE(data->xml_dom); VIR_FREE(data->xml_dom);
if (data->vm) {
priv = data->vm->privateData;
priv->mon = NULL;
virObjectUnref(data->vm); virObjectUnref(data->vm);
}
qemuMonitorTestFree(data->mon); qemuMonitorTestFree(data->mon);
VIR_FREE(data); VIR_FREE(data);
} }