From: Michal Privoznik <mprivozn@redhat.com>
Date: Thu, 7 Jul 2011 15:33:15 +0000 (+0200)
Subject: qemu: Don't chown files on NFS share if dynamic_ownership is off
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=724819a10a92a8709f9276521a0cf27016b5c7b2;p=libvirt.git

qemu: Don't chown files on NFS share if dynamic_ownership is off

When dynamic ownership is disabled we don't want to chown any files,
not just local.
---

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 4f12beba91..a05a1ee992 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -2164,11 +2164,10 @@ static int qemudDomainSaveFlag(struct qemud_driver *driver, virDomainPtr dom,
         is_reg = true;
     } else {
         is_reg = !!S_ISREG(sb.st_mode);
-        /* If the path is regular local file which exists
+        /* If the path is regular file which exists
          * already and dynamic_ownership is off, we don't
          * want to change it's ownership, just open it as-is */
-        if (is_reg && !driver->dynamicOwnership &&
-            virStorageFileIsSharedFS(path) == 0) {
+        if (is_reg && !driver->dynamicOwnership) {
             uid=sb.st_uid;
             gid=sb.st_gid;
         }