virsh: don't print error info when snapshot xml is not changed

If xml is not changed, error message is printed.
So check ret value instead of edit variable for error message.

diff --git a/tools/virsh-snapshot.c b/tools/virsh-snapshot.c
index aff91d3a8bc0ad7776c52e02eea96af6d72752e0..5d84f7a5fa9764e5a10e09b037129a26fbde0eba 100644 (file)
--- a/tools/virsh-snapshot.c
+++ b/tools/virsh-snapshot.c
@@ -449,7 +449,7 @@ cmdSnapshotEdit(vshControl *ctl, const vshCmd *cmd)
                  _("Snapshot %s XML configuration not changed.\n"), \
                  name);                                             \
         ret = true;                                                 \
-        goto cleanup;                                               \
+        goto edit_cleanup;                                          \
     }
 #define EDIT_DEFINE \
     (strstr(doc, "<state>disk-snapshot</state>") ? \
@@ -487,10 +487,10 @@ cmdSnapshotEdit(vshControl *ctl, const vshCmd *cmd)
     ret = true;
 
 cleanup:
+    if (!ret)
+        vshError(ctl, _("Failed to update %s"), name);
     if (edited)
         virDomainSnapshotFree(edited);
-    else
-        vshError(ctl, _("Failed to update %s"), name);
     if (snapshot)
         virDomainSnapshotFree(snapshot);
     if (dom)