Commit 0703b36e authored by Al Viro's avatar Al Viro
Browse files

debugfs: remove duplicate checks in callers of start_creating() 



we'd already verified that DEBUGFS_ALLOW_API was there in
start_creating() - it would've failed otherwise

Acked-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent ebb54484

fs/debugfs/inode.c

+0 −15
Original line number Diff line number Diff line
@@ -433,11 +433,6 @@ static struct dentry *__debugfs_create_file(const char *name, umode_t mode, 
	if (IS_ERR(dentry))

		return dentry;



	if (!(debugfs_allow & DEBUGFS_ALLOW_API)) {

		failed_creating(dentry);

		return ERR_PTR(-EPERM);

	}



	inode = debugfs_get_inode(dentry->d_sb);

	if (unlikely(!inode)) {

		pr_err("out of free dentries, can not create file '%s'\n",
@@ -583,11 +578,6 @@ struct dentry *debugfs_create_dir(const char *name, struct dentry *parent) 
	if (IS_ERR(dentry))

		return dentry;



	if (!(debugfs_allow & DEBUGFS_ALLOW_API)) {

		failed_creating(dentry);

		return ERR_PTR(-EPERM);

	}



	inode = debugfs_get_inode(dentry->d_sb);

	if (unlikely(!inode)) {

		pr_err("out of free dentries, can not create directory '%s'\n",
@@ -630,11 +620,6 @@ struct dentry *debugfs_create_automount(const char *name, 
	if (IS_ERR(dentry))

		return dentry;



	if (!(debugfs_allow & DEBUGFS_ALLOW_API)) {

		failed_creating(dentry);

		return ERR_PTR(-EPERM);

	}



	inode = debugfs_get_inode(dentry->d_sb);

	if (unlikely(!inode)) {

		pr_err("out of free dentries, can not create automount '%s'\n",