External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-10-03 12:55:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

virsh: Fix return code for dump

Browse Source 
After the commit dc0771c, ret variable no longer
represents the status of the return code, use
data->ret replace it.

Signed-off-by: Xu Yandong <xuyandong2@huawei.com>
Reviewed-by: Andrea Bolognani <abologna@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Xu Yandong 2020-05-04 16:36:19 +08:00 committed by Andrea Bolognani
parent 7b0f46343c
commit 5534e4e659
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tools/virsh-domain.c
View File

@ -5522,7 +5522,6 @@ static bool
cmdDump(vshControl *ctl, const vshCmd *cmd) cmdDump(vshControl *ctl, const vshCmd *cmd)
{ {
virDomainPtr dom; virDomainPtr dom;
bool ret = false;
bool verbose = false; bool verbose = false;
const char *name = NULL; const char *name = NULL;
const char *to = NULL; const char *to = NULL;
@ -5556,12 +5555,12 @@ cmdDump(vshControl *ctl, const vshCmd *cmd)
virThreadJoin(&workerThread); virThreadJoin(&workerThread);
if (!ret) if (!data.ret)
vshPrintExtra(ctl, _("\nDomain %s dumped to %s\n"), name, to); vshPrintExtra(ctl, _("\nDomain %s dumped to %s\n"), name, to);
cleanup: cleanup:
virshDomainFree(dom); virshDomainFree(dom);
return !ret; return !data.ret;
} }
static const vshCmdInfo info_screenshot[] = { static const vshCmdInfo info_screenshot[] = {