From a20e8bcad5c17380b52cc3c2469660eb63a78304 Mon Sep 17 00:00:00 2001 From: Martin Kletzander Date: Wed, 25 Jan 2017 09:35:47 +0100 Subject: [PATCH] virsh: Actually make blkdeviotune --group_name work Function vshCommandOptStringReq() returns -1 on error and 0 on success. The code, however, used the 'group_name' variable only if it returned 1 (never). Signed-off-by: Martin Kletzander --- tools/virsh-domain.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index 93587e8bc7..4360306989 100644 --- a/tools/virsh-domain.c +++ b/tools/virsh-domain.c @@ -1398,11 +1398,12 @@ cmdBlkdeviotune(vshControl *ctl, const vshCmd *cmd) VSH_ADD_IOTUNE(write-iops-sec-max-length, WRITE_IOPS_SEC_MAX_LENGTH); #undef VSH_ADD_IOTUNE - rv = vshCommandOptStringReq(ctl, cmd, "group_name", &group_name); - if (rv < 0) { + if (vshCommandOptStringReq(ctl, cmd, "group_name", &group_name) < 0) { vshError(ctl, "%s", _("Unable to parse group parameter")); goto cleanup; - } else if (rv > 0) { + } + + if (group_name) { if (virTypedParamsAddString(¶ms, &nparams, &maxparams, VIR_DOMAIN_BLOCK_IOTUNE_GROUP_NAME, group_name) < 0)