Fix loop ownership

We changed the symlink ownership instead of the target device.
danielerez · Jul 24, 2019 · fda52be · fda52be
1 parent 91004a4
commit fda52be
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/userstorage/loop.py b/userstorage/loop.py
@@ -54,15 +54,15 @@ def create(self):
             raise backend.CreateFailed(
                 "Error creating loop device: {}".format(e))
 
+        if os.geteuid() != 0:
+            osutil.chown(device)
+
         # Remove stale symlink.
         if os.path.islink(self.path):
             os.unlink(self.path)
 
         os.symlink(device, self.path)
 
-        if os.geteuid() != 0:
-            osutil.chown(self.path)
-
     def delete(self):
         log.info("Removing loop device %s", self.path)
         if self.exists():