From: Miklos Szeredi Date: Wed, 15 Jun 2016 12:18:59 +0000 (+0200) Subject: ovl: fix uid/gid when creating over whiteout X-Git-Tag: v4.1.12-108.0.20170806_1300~39 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=13a40a5f1c27872d2234ec0a34695fb064277006;p=users%2Fjedix%2Flinux-maple.git ovl: fix uid/gid when creating over whiteout Fix a regression when creating a file over a whiteout. The new file/directory needs to use the current fsuid/fsgid, not the ones from the mounter's credentials. The refcounting is a bit tricky: prepare_creds() sets an original refcount, override_creds() gets one more, which revert_cred() drops. So 1) we need to expicitly put the mounter's credentials when overriding with the updated one 2) we need to put the original ref to the updated creds (and this can safely be done before revert_creds(), since we'll still have the ref from override_creds()). Reported-by: Stephen Smalley Fixes: 3fe6e52f0626 ("ovl: override creds with the ones from the superblock mounter") Signed-off-by: Miklos Szeredi Orabug: 26401569 (backport upstream commit d0e13f5bbe4be7c8f27736fc40503dcec04b7de0) Signed-off-by: Shan Hai Reviewed-by: Somasundaram Krishnasamy --- diff --git a/fs/overlayfs/dir.c b/fs/overlayfs/dir.c index f61cf9bdbd46d..6a5f8b021aa54 100644 --- a/fs/overlayfs/dir.c +++ b/fs/overlayfs/dir.c @@ -409,12 +409,21 @@ static int ovl_create_or_link(struct dentry *dentry, int mode, dev_t rdev, err = ovl_create_upper(dentry, inode, &stat, link, hardlink); } else { const struct cred *old_cred; + struct cred *override_cred; old_cred = ovl_override_creds(dentry->d_sb); - err = ovl_create_over_whiteout(dentry, inode, &stat, link, - hardlink); + err = -ENOMEM; + override_cred = prepare_creds(); + if (override_cred) { + override_cred->fsuid = old_cred->fsuid; + override_cred->fsgid = old_cred->fsgid; + put_cred(override_creds(override_cred)); + put_cred(override_cred); + err = ovl_create_over_whiteout(dentry, inode, &stat, + link, hardlink); + } revert_creds(old_cred); }