File libvirt-virsh-snapshot-Fix-XPath-query-to-determine-snapshot-state.patch of Package libvirt

From b543cab7e4112b3735678a0c150c898e13492b01 Mon Sep 17 00:00:00 2001
Message-Id: <b543cab7e4112b3735678a0c150c898e13492b01@dist-git>
From: Peter Krempa <pkrempa@redhat.com>
Date: Tue, 8 Apr 2014 11:45:49 +0200
Subject: [PATCH] virsh-snapshot: Fix XPath query to determine snapshot state

https://bugzilla.redhat.com/show_bug.cgi?id=890457

The query didn't match the external state correctly for offline internal
snapshots.

(cherry picked from commit e06816289ced9e34a88e56c371eee920047e30df)

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
---
 tools/virsh-snapshot.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/virsh-snapshot.c b/tools/virsh-snapshot.c
index 87e89cc..0a7e2fe 100644
--- a/tools/virsh-snapshot.c
+++ b/tools/virsh-snapshot.c
@@ -865,8 +865,8 @@ cmdSnapshotInfo(vshControl *ctl, const vshCmd *cmd)
      * external snapshot.  */
     switch (virXPathBoolean("boolean(/domainsnapshot/memory)", ctxt)) {
     case 1:
-        external = virXPathBoolean("boolean(/domainsnapshot/memory/@snapshot=external "
-                                   "| /domainsnapshot/disks/disk/@snapshot=external)",
+        external = virXPathBoolean("boolean(/domainsnapshot/memory[@snapshot='external'] "
+                                   "| /domainsnapshot/disks/disk[@snapshot='external'])",
                                    ctxt);
         break;
     case 0:
-- 
1.9.2

openSUSE Build Service is sponsored by