Skip to content

Commit 89a11f0

Browse files
committed
ovl: drop ovl_setup_cred_for_create()
It is now unused and can be removed. Link: https://patch.msgid.link/20251117-work-ovl-cred-guard-prepare-v2-6-bd1c97a36d7b@kernel.org Reviewed-by: Amir Goldstein <amir73il@gmail.com> Signed-off-by: Christian Brauner <brauner@kernel.org>
1 parent e566bff commit 89a11f0

1 file changed

Lines changed: 0 additions & 33 deletions

File tree

fs/overlayfs/dir.c

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -617,39 +617,6 @@ DEFINE_CLASS(ovl_override_creator_creds,
617617
ovl_override_creator_creds(dentry, inode, mode),
618618
struct dentry *dentry, struct inode *inode, umode_t mode)
619619

620-
static const __maybe_unused struct cred *ovl_setup_cred_for_create(struct dentry *dentry,
621-
struct inode *inode,
622-
umode_t mode,
623-
const struct cred *old_cred)
624-
{
625-
int err;
626-
struct cred *override_cred;
627-
628-
override_cred = prepare_creds();
629-
if (!override_cred)
630-
return ERR_PTR(-ENOMEM);
631-
632-
override_cred->fsuid = inode->i_uid;
633-
override_cred->fsgid = inode->i_gid;
634-
err = security_dentry_create_files_as(dentry, mode, &dentry->d_name,
635-
old_cred, override_cred);
636-
if (err) {
637-
put_cred(override_cred);
638-
return ERR_PTR(err);
639-
}
640-
641-
/*
642-
* Caller is going to match this with revert_creds() and drop
643-
* referenec on the returned creds.
644-
* We must be called with creator creds already, otherwise we risk
645-
* leaking creds.
646-
*/
647-
old_cred = override_creds(override_cred);
648-
WARN_ON_ONCE(old_cred != ovl_creds(dentry->d_sb));
649-
650-
return override_cred;
651-
}
652-
653620
static int ovl_create_handle_whiteouts(struct dentry *dentry,
654621
struct inode *inode,
655622
struct ovl_cattr *attr)

0 commit comments

Comments
 (0)