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 | |
| parent | 3850aba74873aa47fefe6900b99f42f5e656a6e7 (diff) | |
| download | op-kernel-dev-da01636a6511c3bd0c1cf546c47b8e92a837a613.zip op-kernel-dev-da01636a6511c3bd0c1cf546c47b8e92a837a613.tar.gz | |
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>
| -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 8addfe3..d22cd168 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; } |
