From f209d40a7e74e7e53a02c0c7ed20be218d390754 Mon Sep 17 00:00:00 2001 From: Yi Li Date: Wed, 3 Feb 2021 14:18:34 +0800 Subject: [PATCH] qemuDomainAttachRedirdevDevice: Remove need_release variable Get rid of the 'need_release' variable. The code can be rewritten so that it is not needed. Signed-off-by: Yi Li Reviewed-by: Daniel Henrique Barboza Reviewed-by: Michal Privoznik --- src/qemu/qemu_hotplug.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index 213be41553..94c3df6368 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -1924,18 +1924,16 @@ int qemuDomainAttachRedirdevDevice(virQEMUDriverPtr driver, bool chardevAdded = false; g_autofree char *tlsAlias = NULL; const char *secAlias = NULL; - bool need_release = false; virErrorPtr orig_err; if (qemuAssignDeviceRedirdevAlias(def, redirdev, -1) < 0) - goto cleanup; + return -1; if (!(charAlias = qemuAliasChardevFromDevAlias(redirdev->info.alias))) - goto cleanup; + return -1; if ((virDomainUSBAddressEnsure(priv->usbaddrs, &redirdev->info)) < 0) - goto cleanup; - need_release = true; + return -1; if (!(devstr = qemuBuildRedirdevDevStr(def, redirdev, priv->qemuCaps))) goto cleanup; @@ -1967,7 +1965,7 @@ int qemuDomainAttachRedirdevDevice(virQEMUDriverPtr driver, audit: virDomainAuditRedirdev(vm, redirdev, "attach", ret == 0); cleanup: - if (ret < 0 && need_release) + if (ret < 0) qemuDomainReleaseDeviceAddress(vm, &redirdev->info); return ret;