[libvirt] [PATCH] qemu: Don't chown files on NFS share if dynamic_ownership is off

Michal Privoznik mprivozn at redhat.com
Thu Jul 7 15:33:15 UTC 2011


When dynamic ownership is disabled we don't want to chown any files,
not just local.
---
 src/qemu/qemu_driver.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 52b7dfd..968865f 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -2163,11 +2163,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;
         }
-- 
1.7.5.rc3




More information about the libvir-list mailing list