ovl: copy up file size as well

Copy i_size of the underlying inode to the overlay inode in ovl_copyattr().

This is in preparation for stacking I/O operations on overlay files.

This patch shouldn't have any observable effect.

Remove stale comment from ovl_setattr() [spotted by Vivek Goyal].

Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>

1 files changed, 0 insertions, 9 deletions

diff --git a/fs/overlayfs/inode.c b/fs/overlayfs/inode.c
index 3f1e5bd0ba87..5b6c86703d34 100644
--- a/fs/overlayfs/inode.c
+++ b/fs/overlayfs/inode.c
@@ -22,15 +22,6 @@ int ovl_setattr(struct dentry *dentry, struct iattr *attr)
         struct dentry *upperdentry;
         const struct cred *old_cred;
 
-        /*
-         * Check for permissions before trying to copy-up.  This is redundant
-         * since it will be rechecked later by ->setattr() on upper dentry.  But
-         * without this, copy-up can be triggered by just about anybody.
-         *
-         * We don't initialize inode->size, which just means that
-         * inode_newsize_ok() will always check against MAX_LFS_FILESIZE and not
-         * check for a swapfile (which this won't be anyway).
-         */
         err = setattr_prepare(dentry, attr);
         if (err)
                 return err;