summaryrefslogtreecommitdiffstats
path: root/fs/open.c
diff options
context:
space:
mode:
authorTetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>2011-01-19 21:08:41 +0900
committerAl Viro <viro@zeniv.linux.org.uk>2011-03-21 01:10:41 -0400
commitc212f9aaf9101a037fb7f59e75e639437e11d758 (patch)
treeda00b3e84fa5ffc7648c14593aaf1cbb4931c229 /fs/open.c
parentaa597bc1f9476d0527e35d6dd9b481422e8205a0 (diff)
downloadlinux-c212f9aaf9101a037fb7f59e75e639437e11d758.tar.gz
linux-c212f9aaf9101a037fb7f59e75e639437e11d758.tar.bz2
linux-c212f9aaf9101a037fb7f59e75e639437e11d758.zip
fs: Use BUG_ON(!mnt) at dentry_open().
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>
Diffstat (limited to 'fs/open.c')
-rw-r--r--fs/open.c13
1 files changed, 2 insertions, 11 deletions
diff --git a/fs/open.c b/fs/open.c
index f83ca80cc59a..b52cf013ffa1 100644
--- 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();