[PATCH 08/12] qemuMonitorAddObject: Refactor cleanup
Remove freeing/clearing of @props as the function doesn't guarantee that
it happens on success, rename the variable hodling copy of the alias and
use g_autofree to automatically free it and remove the cleanup label as
well as 'ret' variable.
Signed-off-by: Peter Krempa <pkrempa(a)redhat.com>
---
src/qemu/qemu_monitor.c | 23 ++++++++---------------
1 file changed, 8 insertions(+), 15 deletions(-)
diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
index 7919eb3a04..55ec032b5f 100644
--- a/src/qemu/qemu_monitor.c
+++ b/src/qemu/qemu_monitor.c
@@ -3050,13 +3050,12 @@ qemuMonitorAddObject(qemuMonitorPtr mon,
{
const char *type = NULL;
const char *id = NULL;
- char *tmp = NULL;
- int ret = -1;
+ g_autofree char *aliasCopy = NULL;
if (!*props) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("object props can't be NULL"));
- goto cleanup;
+ return -1;
}
type = virJSONValueObjectGetString(*props, "qom-type");
@@ -3064,31 +3063,25 @@ qemuMonitorAddObject(qemuMonitorPtr mon,
VIR_DEBUG("type=%s id=%s", NULLSTR(type), NULLSTR(id));
- QEMU_CHECK_MONITOR_GOTO(mon, cleanup);
+ QEMU_CHECK_MONITOR(mon);
if (!id || !type) {
virReportError(VIR_ERR_INTERNAL_ERROR,
_("missing alias or qom-type for qemu object
'%s'"),
NULLSTR(type));
- goto cleanup;
+ return -1;
}
if (alias)
- tmp = g_strdup(id);
+ aliasCopy = g_strdup(id);
if (qemuMonitorJSONAddObject(mon, props) < 0)
- goto cleanup;
+ return -1;
if (alias)
- *alias = g_steal_pointer(&tmp);
-
- ret = 0;
+ *alias = g_steal_pointer(&aliasCopy);
- cleanup:
- VIR_FREE(tmp);
- virJSONValueFree(*props);
- *props = NULL;
- return ret;
+ return 0;
}
--
2.29.2