Commit a59d6293 authored May 23, 2012 by Arend van Spriel Committed by Greg Kroah-Hartman Jun 13, 2012

debugfs: change parameter check in debugfs_remove() functions

The dentry parameter in debugfs_remove() and debugfs_remove_recursive()
is checked being a NULL pointer. To make cleanup by callers easier this
check is extended using the IS_ERR_OR_NULL macro instead because the
debugfs_create_... functions can return a ERR_PTR() value.

Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 0998d063

fs/debugfs/inode.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -498,7 +498,7 @@ void debugfs_remove(struct dentry *dentry)
		struct dentry *parent;
		int ret;

		if (!dentry)
		if (IS_ERR_OR_NULL(dentry))
		return;

		parent = dentry->d_parent;
		@@ -530,7 +530,7 @@ void debugfs_remove_recursive(struct dentry *dentry)
		struct dentry *child;
		struct dentry *parent;

		if (!dentry)
		if (IS_ERR_OR_NULL(dentry))
		return;

		parent = dentry->d_parent;