virsh: Don't read nparams when virDomainGetMemoryParameters fails

Also exit early when nparams is 0.

diff --git a/tools/virsh.c b/tools/virsh.c
index d1ef6981ca61e4ccc1b17c3aed5787d29c352f42..3e37b063293eade9e225b8e5a394b617e306644b 100644 (file)
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -2941,17 +2941,22 @@ cmdMemtune(vshControl * ctl, const vshCmd * cmd)
 
     if (nparams == 0) {
         /* get the number of memory parameters */
-        if ((virDomainGetMemoryParameters(dom, NULL, &nparams, 0) != 0) &&
-            (nparams != 0)) {
+        if (virDomainGetMemoryParameters(dom, NULL, &nparams, 0) != 0) {
             vshError(ctl, "%s",
                      _("Unable to get number of memory parameters"));
             goto cleanup;
         }
 
+        if (nparams == 0) {
+            /* nothing to output */
+            ret = TRUE;
+            goto cleanup;
+        }
+
         /* now go get all the memory parameters */
         params = vshMalloc(ctl, sizeof(virMemoryParameter) * nparams);
         memset(params, 0, sizeof(virMemoryParameter) * nparams);
-        if (virDomainGetMemoryParameters(dom, params, &nparams, 0)) {
+        if (virDomainGetMemoryParameters(dom, params, &nparams, 0) != 0) {
             vshError(ctl, "%s", _("Unable to get memory parameters"));
             goto cleanup;
         }
@@ -3026,7 +3031,7 @@ cmdMemtune(vshControl * ctl, const vshCmd * cmd)
             }
         }
         if (virDomainSetMemoryParameters(dom, params, nparams, 0) != 0)
-            vshError(ctl, "%s", _("Unable to change Memory Parameters"));
+            vshError(ctl, "%s", _("Unable to change memory parameters"));
         else
             ret = TRUE;
     }