diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2012-01-09 01:45:28 +0100 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-01-09 02:19:12 +0100 |
| commit | da01636a6511c3bd0c1cf546c47b8e92a837a613 (patch) | |
| tree | 394f40f4763d444ace4990ff1a5d21b386f83167 /fs/exofs/super.c | |
| parent | devpts: fix double-free on mount failure (diff) | |
| download | linux-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 '')
| -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) 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; } |