fs: Use BUG_ON(!mnt) at dentry_open().
authorTetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Wed, 19 Jan 2011 12:08:41 +0000 (21:08 +0900)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 21 Mar 2011 05:10:41 +0000 (01:10 -0400)
dentry_open() requires callers to pass a valid vfsmount.

Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/open.c

index f83ca80cc59a1d72c8b933d7562efc853cfe3f8c..b52cf013ffa145de32e5c641aba7a3625de8cc29 100644 (file)
--- a/fs/open.c
+++ b/fs/open.c
@@ -835,17 +835,8 @@ struct file *dentry_open(struct dentry *dentry, struct vfsmount *mnt, int flags,
 
        validate_creds(cred);
 
-       /*
-        * We must always pass in a valid mount pointer.   Historically
-        * callers got away with not passing it, but we must enforce this at
-        * the earliest possible point now to avoid strange problems deep in the
-        * filesystem stack.
-        */
-       if (!mnt) {
-               printk(KERN_WARNING "%s called with NULL vfsmount\n", __func__);
-               dump_stack();
-               return ERR_PTR(-EINVAL);
-       }
+       /* We must always pass in a valid mount pointer. */
+       BUG_ON(!mnt);
 
        error = -ENFILE;
        f = get_empty_filp();