virsh: remove variable 'ret' and 'inactive'

Signed-off-by: Kristina Hanicova <khanicov@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Kristina Hanicova 2021-09-24 01:49:10 +02:00 committed by Michal Privoznik
parent 409ccebcaa
commit 45b17385e7
2 changed files with 12 additions and 20 deletions

View File

@ -485,26 +485,23 @@ static bool
cmdInterfaceDumpXML(vshControl *ctl, const vshCmd *cmd) cmdInterfaceDumpXML(vshControl *ctl, const vshCmd *cmd)
{ {
virInterfacePtr iface; virInterfacePtr iface;
bool ret = true;
g_autofree char *dump = NULL; g_autofree char *dump = NULL;
unsigned int flags = 0; unsigned int flags = 0;
bool inactive = vshCommandOptBool(cmd, "inactive");
if (inactive) if (vshCommandOptBool(cmd, "inactive"))
flags |= VIR_INTERFACE_XML_INACTIVE; flags |= VIR_INTERFACE_XML_INACTIVE;
if (!(iface = virshCommandOptInterface(ctl, cmd, NULL))) if (!(iface = virshCommandOptInterface(ctl, cmd, NULL)))
return false; return false;
dump = virInterfaceGetXMLDesc(iface, flags); if (!(dump = virInterfaceGetXMLDesc(iface, flags))) {
if (dump != NULL) { virInterfaceFree(iface);
vshPrint(ctl, "%s", dump); return false;
} else {
ret = false;
} }
vshPrint(ctl, "%s", dump);
virInterfaceFree(iface); virInterfaceFree(iface);
return ret; return true;
} }
/* /*

View File

@ -360,28 +360,23 @@ static bool
cmdNetworkDumpXML(vshControl *ctl, const vshCmd *cmd) cmdNetworkDumpXML(vshControl *ctl, const vshCmd *cmd)
{ {
virNetworkPtr network; virNetworkPtr network;
bool ret = true;
g_autofree char *dump = NULL; g_autofree char *dump = NULL;
unsigned int flags = 0; unsigned int flags = 0;
int inactive;
if (!(network = virshCommandOptNetwork(ctl, cmd, NULL))) if (!(network = virshCommandOptNetwork(ctl, cmd, NULL)))
return false; return false;
inactive = vshCommandOptBool(cmd, "inactive"); if (vshCommandOptBool(cmd, "inactive"))
if (inactive)
flags |= VIR_NETWORK_XML_INACTIVE; flags |= VIR_NETWORK_XML_INACTIVE;
dump = virNetworkGetXMLDesc(network, flags); if (!(dump = virNetworkGetXMLDesc(network, flags))) {
virNetworkFree(network);
if (dump != NULL) { return false;
vshPrint(ctl, "%s", dump);
} else {
ret = false;
} }
vshPrint(ctl, "%s", dump);
virNetworkFree(network); virNetworkFree(network);
return ret; return true;
} }
/* /*