Messages in this thread Patch in this message | ![/](/images/icornerl.gif) | | Date | Mon, 25 Apr 2005 22:12:42 +0200 | From | Colin Leroy <> | Subject | Re: [PATCH] hfsplus: don't oops on bad FS |
| |
On 25 Apr 2005 at 21h04, Christoph Hellwig wrote:
Hi,
> On Mon, Apr 25, 2005 at 10:03:45PM +0200, Colin Leroy wrote: > > absolutely no need to cast here.
Third update then :-)
Signed-off-by: Colin Leroy <colin@colino.net> --- a/fs/hfsplus/super.c 2005-04-25 21:56:56.000000000 +0200 +++ b/fs/hfsplus/super.c 2005-04-25 21:58:39.000000000 +0200 @@ -226,6 +226,9 @@ brelse(HFSPLUS_SB(sb).s_vhbh); if (HFSPLUS_SB(sb).nls) unload_nls(HFSPLUS_SB(sb).nls); + + kfree(sb->s_fs_info); + sb->s_fs_info = NULL; } static int hfsplus_statfs(struct super_block *sb, struct kstatfs *buf) @@ -298,7 +301,7 @@ if (!silent) printk("HFS+-fs: unable to parse mount options\n"); err = -EINVAL; - goto cleanup; + goto cleanup_little; } /* temporarily use utf8 to correctly find the hidden dir below */ @@ -307,7 +310,7 @@ if (!nls) { printk("HFS+: unable to load nls for utf8\n"); err = -EINVAL; - goto cleanup; + goto cleanup_little; } /* Grab the volume header */ @@ -315,7 +318,7 @@ if (!silent) printk("HFS+-fs: unable to find HFS+ superblock\n"); err = -EINVAL; - goto cleanup; + goto cleanup_little; } vhdr = HFSPLUS_SB(sb).s_vhdr; @@ -428,8 +431,12 @@ cleanup: hfsplus_put_super(sb); + +cleanup_little: if (nls) unload_nls(nls); + sb->s_fs_info = NULL; + kfree(sbi); return err; } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |