diff options
Diffstat (limited to 'fs/hfs')
-rw-r--r-- | fs/hfs/mdb.c | 5 | ||||
-rw-r--r-- | fs/hfs/super.c | 8 |
2 files changed, 8 insertions, 5 deletions
diff --git a/fs/hfs/mdb.c b/fs/hfs/mdb.c index 4efb640c4d0c..217e32f37e0b 100644 --- a/fs/hfs/mdb.c +++ b/fs/hfs/mdb.c | |||
@@ -333,6 +333,8 @@ void hfs_mdb_close(struct super_block *sb) | |||
333 | * Release the resources associated with the in-core MDB. */ | 333 | * Release the resources associated with the in-core MDB. */ |
334 | void hfs_mdb_put(struct super_block *sb) | 334 | void hfs_mdb_put(struct super_block *sb) |
335 | { | 335 | { |
336 | if (!HFS_SB(sb)) | ||
337 | return; | ||
336 | /* free the B-trees */ | 338 | /* free the B-trees */ |
337 | hfs_btree_close(HFS_SB(sb)->ext_tree); | 339 | hfs_btree_close(HFS_SB(sb)->ext_tree); |
338 | hfs_btree_close(HFS_SB(sb)->cat_tree); | 340 | hfs_btree_close(HFS_SB(sb)->cat_tree); |
@@ -340,4 +342,7 @@ void hfs_mdb_put(struct super_block *sb) | |||
340 | /* free the buffers holding the primary and alternate MDBs */ | 342 | /* free the buffers holding the primary and alternate MDBs */ |
341 | brelse(HFS_SB(sb)->mdb_bh); | 343 | brelse(HFS_SB(sb)->mdb_bh); |
342 | brelse(HFS_SB(sb)->alt_mdb_bh); | 344 | brelse(HFS_SB(sb)->alt_mdb_bh); |
345 | |||
346 | kfree(HFS_SB(sb)); | ||
347 | sb->s_fs_info = NULL; | ||
343 | } | 348 | } |
diff --git a/fs/hfs/super.c b/fs/hfs/super.c index 1e2c193134cc..ab783f6afa3b 100644 --- a/fs/hfs/super.c +++ b/fs/hfs/super.c | |||
@@ -297,7 +297,7 @@ static int hfs_fill_super(struct super_block *sb, void *data, int silent) | |||
297 | res = -EINVAL; | 297 | res = -EINVAL; |
298 | if (!parse_options((char *)data, sbi)) { | 298 | if (!parse_options((char *)data, sbi)) { |
299 | hfs_warn("hfs_fs: unable to parse mount options.\n"); | 299 | hfs_warn("hfs_fs: unable to parse mount options.\n"); |
300 | goto bail3; | 300 | goto bail; |
301 | } | 301 | } |
302 | 302 | ||
303 | sb->s_op = &hfs_super_operations; | 303 | sb->s_op = &hfs_super_operations; |
@@ -310,7 +310,7 @@ static int hfs_fill_super(struct super_block *sb, void *data, int silent) | |||
310 | hfs_warn("VFS: Can't find a HFS filesystem on dev %s.\n", | 310 | hfs_warn("VFS: Can't find a HFS filesystem on dev %s.\n", |
311 | hfs_mdb_name(sb)); | 311 | hfs_mdb_name(sb)); |
312 | res = -EINVAL; | 312 | res = -EINVAL; |
313 | goto bail2; | 313 | goto bail; |
314 | } | 314 | } |
315 | 315 | ||
316 | /* try to get the root inode */ | 316 | /* try to get the root inode */ |
@@ -340,10 +340,8 @@ bail_iput: | |||
340 | iput(root_inode); | 340 | iput(root_inode); |
341 | bail_no_root: | 341 | bail_no_root: |
342 | hfs_warn("hfs_fs: get root inode failed.\n"); | 342 | hfs_warn("hfs_fs: get root inode failed.\n"); |
343 | bail: | ||
343 | hfs_mdb_put(sb); | 344 | hfs_mdb_put(sb); |
344 | bail2: | ||
345 | bail3: | ||
346 | kfree(sbi); | ||
347 | return res; | 345 | return res; |
348 | } | 346 | } |
349 | 347 | ||