render / rpms / libvirt

Forked from rpms/libvirt 9 months ago
Clone
Daniel P. Berrangé bc1b3e
From 2edd63a0dbd445112db23596ee0128521e8f1ff5 Mon Sep 17 00:00:00 2001
Daniel P. Berrangé bc1b3e
From: Jiri Denemark <jdenemar@redhat.com>
Daniel P. Berrangé bc1b3e
Date: Wed, 5 Aug 2020 10:01:45 +0200
Daniel P. Berrangé bc1b3e
Subject: [libvirt PATCH] util: Fix logic in virFileSetCOW
Daniel P. Berrangé bc1b3e
MIME-Version: 1.0
Daniel P. Berrangé bc1b3e
Content-Type: text/plain; charset=UTF-8
Daniel P. Berrangé bc1b3e
Content-Transfer-Encoding: 8bit
Daniel P. Berrangé bc1b3e
Daniel P. Berrangé bc1b3e
When COW is not explicitly requested to be disabled or enabled, the
Daniel P. Berrangé bc1b3e
function is supposed to do nothing on non-BTRFS file systems.
Daniel P. Berrangé bc1b3e
Daniel P. Berrangé bc1b3e
Fixes commit 7230bc95aa78379c9ee20cf59394c5fc4305b75b.
Daniel P. Berrangé bc1b3e
Daniel P. Berrangé bc1b3e
https://bugzilla.redhat.com/show_bug.cgi?id=1866157
Daniel P. Berrangé bc1b3e
Daniel P. Berrangé bc1b3e
Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
Daniel P. Berrangé bc1b3e
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Daniel P. Berrangé bc1b3e
---
Daniel P. Berrangé bc1b3e
 src/util/virfile.c | 2 +-
Daniel P. Berrangé bc1b3e
 1 file changed, 1 insertion(+), 1 deletion(-)
Daniel P. Berrangé bc1b3e
Daniel P. Berrangé bc1b3e
diff --git a/src/util/virfile.c b/src/util/virfile.c
Daniel P. Berrangé bc1b3e
index 6ada3a708c..7f22884b07 100644
Daniel P. Berrangé bc1b3e
--- a/src/util/virfile.c
Daniel P. Berrangé bc1b3e
+++ b/src/util/virfile.c
Daniel P. Berrangé bc1b3e
@@ -4522,7 +4522,7 @@ virFileSetCOW(const char *path,
Daniel P. Berrangé bc1b3e
     }
Daniel P. Berrangé bc1b3e
 
Daniel P. Berrangé bc1b3e
     if (buf.f_type != BTRFS_SUPER_MAGIC) {
Daniel P. Berrangé bc1b3e
-        if (state == VIR_TRISTATE_BOOL_ABSENT) {
Daniel P. Berrangé bc1b3e
+        if (state != VIR_TRISTATE_BOOL_ABSENT) {
Daniel P. Berrangé bc1b3e
             virReportSystemError(ENOSYS,
Daniel P. Berrangé bc1b3e
                                  _("unable to control COW flag on '%s', not btrfs"),
Daniel P. Berrangé bc1b3e
                                  path);
Daniel P. Berrangé bc1b3e
-- 
Daniel P. Berrangé bc1b3e
2.26.2
Daniel P. Berrangé bc1b3e