提交 3bc41601 编写于 作者: J Jiri Denemark

qemu: Make qemuMigrationParamsFree follow common pattern

Our *Free functions usually do not take a double pointer and the caller
has to make sure it doesn't use the stale pointer after the *Free
function returns.
Signed-off-by: NJiri Denemark <jdenemar@redhat.com>
Reviewed-by: NJán Tomko <jtomko@redhat.com>
上级 1b100b74
......@@ -12759,7 +12759,7 @@ qemuDomainMigratePerform3Params(virDomainPtr dom,
flags, dname, bandwidth, true);
cleanup:
VIR_FREE(compression);
qemuMigrationParamsFree(&migParams);
qemuMigrationParamsFree(migParams);
VIR_FREE(migrate_disks);
return ret;
}
......
......@@ -50,13 +50,13 @@ qemuMigrationParamsClear(qemuMonitorMigrationParamsPtr migParams)
void
qemuMigrationParamsFree(qemuMonitorMigrationParamsPtr *migParams)
qemuMigrationParamsFree(qemuMonitorMigrationParamsPtr migParams)
{
if (!*migParams)
if (!migParams)
return;
qemuMigrationParamsClear(*migParams);
VIR_FREE(*migParams);
qemuMigrationParamsClear(migParams);
VIR_FREE(migParams);
}
......@@ -101,7 +101,7 @@ qemuMigrationParamsFromFlags(virTypedParameterPtr params,
return migParams;
error:
qemuMigrationParamsFree(&migParams);
qemuMigrationParamsFree(migParams);
return NULL;
}
......
......@@ -37,7 +37,7 @@ void
qemuMigrationParamsClear(qemuMonitorMigrationParamsPtr migParams);
void
qemuMigrationParamsFree(qemuMonitorMigrationParamsPtr *migParams);
qemuMigrationParamsFree(qemuMonitorMigrationParamsPtr migParams);
int
qemuMigrationParamsSet(virQEMUDriverPtr driver,
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册