diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2012-01-08 19:45:28 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-01-08 20:19:12 -0500 |
commit | da01636a6511c3bd0c1cf546c47b8e92a837a613 (patch) | |
tree | 394f40f4763d444ace4990ff1a5d21b386f83167 /fs/exofs/super.c | |
parent | 3850aba74873aa47fefe6900b99f42f5e656a6e7 (diff) |
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/super.c')
-rw-r--r-- | fs/exofs/super.c | 2 |
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) | |||
838 | ret = bdi_setup_and_register(&sbi->bdi, "exofs", BDI_CAP_MAP_COPY); | 838 | ret = bdi_setup_and_register(&sbi->bdi, "exofs", BDI_CAP_MAP_COPY); |
839 | if (ret) { | 839 | if (ret) { |
840 | EXOFS_DBGMSG("Failed to bdi_setup_and_register\n"); | 840 | EXOFS_DBGMSG("Failed to bdi_setup_and_register\n"); |
841 | dput(sb->s_root); | ||
842 | sb->s_root = NULL; | ||
841 | goto free_sbi; | 843 | goto free_sbi; |
842 | } | 844 | } |
843 | 845 | ||