util: midonet: use g_auto for virCommand

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Ján Tomko 2022-01-17 18:02:28 +01:00
parent a269fae099
commit aa9cd08d92

View File

@ -39,8 +39,7 @@ int
virNetDevMidonetBindPort(const char *ifname, virNetDevMidonetBindPort(const char *ifname,
const virNetDevVPortProfile *virtualport) const virNetDevVPortProfile *virtualport)
{ {
int ret = -1; g_autoptr(virCommand) cmd = NULL;
virCommand *cmd = NULL;
char virtportuuid[VIR_UUID_STRING_BUFLEN]; char virtportuuid[VIR_UUID_STRING_BUFLEN];
virUUIDFormat(virtualport->interfaceID, virtportuuid); virUUIDFormat(virtualport->interfaceID, virtportuuid);
@ -53,13 +52,10 @@ virNetDevMidonetBindPort(const char *ifname,
virReportError(VIR_ERR_INTERNAL_ERROR, virReportError(VIR_ERR_INTERNAL_ERROR,
_("Unable to bind port %s to the virtual port %s"), _("Unable to bind port %s to the virtual port %s"),
ifname, virtportuuid); ifname, virtportuuid);
goto cleanup; return -1;
} }
ret = 0; return 0;
cleanup:
virCommandFree(cmd);
return ret;
} }
/** /**
@ -73,8 +69,7 @@ virNetDevMidonetBindPort(const char *ifname,
int int
virNetDevMidonetUnbindPort(const virNetDevVPortProfile *virtualport) virNetDevMidonetUnbindPort(const virNetDevVPortProfile *virtualport)
{ {
int ret = -1; g_autoptr(virCommand) cmd = NULL;
virCommand *cmd = NULL;
char virtportuuid[VIR_UUID_STRING_BUFLEN]; char virtportuuid[VIR_UUID_STRING_BUFLEN];
virUUIDFormat(virtualport->interfaceID, virtportuuid); virUUIDFormat(virtualport->interfaceID, virtportuuid);
@ -86,11 +81,8 @@ virNetDevMidonetUnbindPort(const virNetDevVPortProfile *virtualport)
virReportError(VIR_ERR_INTERNAL_ERROR, virReportError(VIR_ERR_INTERNAL_ERROR,
_("Unable to unbind the virtual port %s from Midonet"), _("Unable to unbind the virtual port %s from Midonet"),
virtportuuid); virtportuuid);
goto cleanup; return -1;
} }
ret = 0; return 0;
cleanup:
virCommandFree(cmd);
return ret;
} }