eCryptfs: Copy up lower inode attrs after setting lower xattr
authorTyler Hicks <tyhicks@canonical.com>
Tue, 7 Feb 2012 23:55:40 +0000 (17:55 -0600)
committerTyler Hicks <tyhicks@canonical.com>
Thu, 16 Feb 2012 22:06:27 +0000 (16:06 -0600)
After passing through a ->setxattr() call, eCryptfs needs to copy the
inode attributes from the lower inode to the eCryptfs inode, as they
may have changed in the lower filesystem's ->setxattr() path.

One example is if an extended attribute containing a POSIX Access
Control List is being set. The new ACL may cause the lower filesystem to
modify the mode of the lower inode and the eCryptfs inode would need to
be updated to reflect the new mode.

https://launchpad.net/bugs/926292

Signed-off-by: Tyler Hicks <tyhicks@canonical.com>
Reported-by: Sebastien Bacher <seb128@ubuntu.com>
Cc: John Johansen <john.johansen@canonical.com>
Cc: <stable@vger.kernel.org>
fs/ecryptfs/inode.c

index 19892d7d2ed1122547536d5bee21cff9293e7a8a..ab35b113003b900ad3592217d64e6cef82fe8f9d 100644 (file)
@@ -1085,6 +1085,8 @@ ecryptfs_setxattr(struct dentry *dentry, const char *name, const void *value,
        }
 
        rc = vfs_setxattr(lower_dentry, name, value, size, flags);
+       if (!rc)
+               fsstack_copy_attr_all(dentry->d_inode, lower_dentry->d_inode);
 out:
        return rc;
 }