提交 461ddf50 编写于 作者: D Daniel Henrique Barboza 提交者: Ján Tomko

qemu_capabilities.c: modernize virQEMUCapsCacheLookupDefault

Use g_autoptr() in qemuCaps to get rid of a virObjectUnref call,
a 'cleanup' label and the 'ret' pointer.
Signed-off-by: NDaniel Henrique Barboza <danielhb413@gmail.com>
Reviewed-by: NJán Tomko <jtomko@redhat.com>
Signed-off-by: NJán Tomko <jtomko@redhat.com>
上级 c508c94d
...@@ -5737,8 +5737,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache, ...@@ -5737,8 +5737,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache,
virArch hostarch = virArchFromHost(); virArch hostarch = virArchFromHost();
virArch arch = hostarch; virArch arch = hostarch;
virDomainVirtType capsType; virDomainVirtType capsType;
virQEMUCapsPtr qemuCaps = NULL; g_autoptr(virQEMUCaps) qemuCaps = NULL;
virQEMUCapsPtr ret = NULL;
virArch arch_from_caps; virArch arch_from_caps;
g_autofree char *probedbinary = NULL; g_autofree char *probedbinary = NULL;
...@@ -5746,14 +5745,14 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache, ...@@ -5746,14 +5745,14 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache,
(virttype = virDomainVirtTypeFromString(virttypeStr)) < 0) { (virttype = virDomainVirtTypeFromString(virttypeStr)) < 0) {
virReportError(VIR_ERR_INVALID_ARG, virReportError(VIR_ERR_INVALID_ARG,
_("unknown virttype: %s"), virttypeStr); _("unknown virttype: %s"), virttypeStr);
goto cleanup; return NULL;
} }
if (archStr && if (archStr &&
(arch = virArchFromString(archStr)) == VIR_ARCH_NONE) { (arch = virArchFromString(archStr)) == VIR_ARCH_NONE) {
virReportError(VIR_ERR_INVALID_ARG, virReportError(VIR_ERR_INVALID_ARG,
_("unknown architecture: %s"), archStr); _("unknown architecture: %s"), archStr);
goto cleanup; return NULL;
} }
if (!binary) { if (!binary) {
...@@ -5762,7 +5761,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache, ...@@ -5762,7 +5761,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache,
} }
if (!(qemuCaps = virQEMUCapsCacheLookup(cache, binary))) if (!(qemuCaps = virQEMUCapsCacheLookup(cache, binary)))
goto cleanup; return NULL;
arch_from_caps = virQEMUCapsGetArch(qemuCaps); arch_from_caps = virQEMUCapsGetArch(qemuCaps);
...@@ -5776,7 +5775,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache, ...@@ -5776,7 +5775,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache,
"match given architecture '%s'"), "match given architecture '%s'"),
virArchToString(arch_from_caps), virArchToString(arch_from_caps),
virArchToString(arch)); virArchToString(arch));
goto cleanup; return NULL;
} }
capsType = virQEMUCapsGetVirtType(qemuCaps); capsType = virQEMUCapsGetVirtType(qemuCaps);
...@@ -5788,7 +5787,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache, ...@@ -5788,7 +5787,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache,
virReportError(VIR_ERR_INVALID_ARG, virReportError(VIR_ERR_INVALID_ARG,
_("KVM is not supported by '%s' on this host"), _("KVM is not supported by '%s' on this host"),
binary); binary);
goto cleanup; return NULL;
} }
if (machine) { if (machine) {
...@@ -5799,7 +5798,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache, ...@@ -5799,7 +5798,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache,
virReportError(VIR_ERR_INVALID_ARG, virReportError(VIR_ERR_INVALID_ARG,
_("the machine '%s' is not supported by emulator '%s'"), _("the machine '%s' is not supported by emulator '%s'"),
machine, binary); machine, binary);
goto cleanup; return NULL;
} }
} else { } else {
machine = virQEMUCapsGetPreferredMachine(qemuCaps, virttype); machine = virQEMUCapsGetPreferredMachine(qemuCaps, virttype);
...@@ -5812,11 +5811,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache, ...@@ -5812,11 +5811,7 @@ virQEMUCapsCacheLookupDefault(virFileCachePtr cache,
if (retMachine) if (retMachine)
*retMachine = machine; *retMachine = machine;
ret = g_steal_pointer(&qemuCaps); return g_steal_pointer(&qemuCaps);
cleanup:
virObjectUnref(qemuCaps);
return ret;
} }
bool bool
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册