summaryrefslogtreecommitdiffstats
path: root/fs/exofs
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2012-01-09 01:45:28 +0100
committerAl Viro <viro@zeniv.linux.org.uk>2012-01-09 02:19:12 +0100
commitda01636a6511c3bd0c1cf546c47b8e92a837a613 (patch)
tree394f40f4763d444ace4990ff1a5d21b386f83167 /fs/exofs
parentdevpts: fix double-free on mount failure (diff)
downloadlinux-da01636a6511c3bd0c1cf546c47b8e92a837a613.tar.xz
linux-da01636a6511c3bd0c1cf546c47b8e92a837a613.zip
exofs: oops after late failure in mount
We have already set ->s_root, so ->put_super() is going to be called. Freeing ->s_fs_info is a bloody bad idea when it's going to be dereferenced very shortly... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/exofs')
-rw-r--r--fs/exofs/super.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/exofs/super.c b/fs/exofs/super.c
index 8addfe314dc7..d22cd168c6ee 100644
--- a/fs/exofs/super.c
+++ b/fs/exofs/super.c
@@ -838,6 +838,8 @@ static int exofs_fill_super(struct super_block *sb, void *data, int silent)
ret = bdi_setup_and_register(&sbi->bdi, "exofs", BDI_CAP_MAP_COPY);
if (ret) {
EXOFS_DBGMSG("Failed to bdi_setup_and_register\n");
+ dput(sb->s_root);
+ sb->s_root = NULL;
goto free_sbi;
}