Skip to content

Commit 0968027

Browse files
author
Miklos Szeredi
committed
ovl: remove upper umask handling from ovl_create_upper()
This is already done by vfs_prepare_mode() when creating the upper object by vfs_create(), vfs_mkdir() and vfs_mknod(). No regressions have been observed in xfstests run with posix acls turned off for the upper filesystem. Fixes: 1639a49 ("fs: move S_ISGID stripping into the vfs_*() helpers") Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
1 parent 9a87907 commit 0968027

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

fs/overlayfs/dir.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -330,9 +330,6 @@ static int ovl_create_upper(struct dentry *dentry, struct inode *inode,
330330
struct dentry *newdentry;
331331
int err;
332332

333-
if (!attr->hardlink && !IS_POSIXACL(udir))
334-
attr->mode &= ~current_umask();
335-
336333
inode_lock_nested(udir, I_MUTEX_PARENT);
337334
newdentry = ovl_create_real(ofs, udir,
338335
ovl_lookup_upper(ofs, dentry->d_name.name,

0 commit comments

Comments
 (0)