diff options
author | Borislav Petkov <bbpetkov@yahoo.de> | 2007-07-16 02:41:45 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-16 12:05:51 -0400 |
commit | 952d9de116ad87261de106464a9eeec038c4cd14 (patch) | |
tree | 415354669cb3b2ccfe7ea066e0bcf79c0a4305de | |
parent | 1f1f642e2f092e37eb9038060eb0100c44f55a11 (diff) |
ext3: fix error handling in ext3_create_journal()
Fix error handling in ext3_create_journal according to kernel conventions.
Signed-off-by: Borislav Petkov <bbpetkov@yahoo.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/ext3/super.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/ext3/super.c b/fs/ext3/super.c index 440c35d98993..8b75f73ba3b4 100644 --- a/fs/ext3/super.c +++ b/fs/ext3/super.c | |||
@@ -2084,6 +2084,7 @@ static int ext3_create_journal(struct super_block * sb, | |||
2084 | unsigned int journal_inum) | 2084 | unsigned int journal_inum) |
2085 | { | 2085 | { |
2086 | journal_t *journal; | 2086 | journal_t *journal; |
2087 | int err; | ||
2087 | 2088 | ||
2088 | if (sb->s_flags & MS_RDONLY) { | 2089 | if (sb->s_flags & MS_RDONLY) { |
2089 | printk(KERN_ERR "EXT3-fs: readonly filesystem when trying to " | 2090 | printk(KERN_ERR "EXT3-fs: readonly filesystem when trying to " |
@@ -2091,13 +2092,15 @@ static int ext3_create_journal(struct super_block * sb, | |||
2091 | return -EROFS; | 2092 | return -EROFS; |
2092 | } | 2093 | } |
2093 | 2094 | ||
2094 | if (!(journal = ext3_get_journal(sb, journal_inum))) | 2095 | journal = ext3_get_journal(sb, journal_inum); |
2096 | if (!journal) | ||
2095 | return -EINVAL; | 2097 | return -EINVAL; |
2096 | 2098 | ||
2097 | printk(KERN_INFO "EXT3-fs: creating new journal on inode %u\n", | 2099 | printk(KERN_INFO "EXT3-fs: creating new journal on inode %u\n", |
2098 | journal_inum); | 2100 | journal_inum); |
2099 | 2101 | ||
2100 | if (journal_create(journal)) { | 2102 | err = journal_create(journal); |
2103 | if (err) { | ||
2101 | printk(KERN_ERR "EXT3-fs: error creating journal.\n"); | 2104 | printk(KERN_ERR "EXT3-fs: error creating journal.\n"); |
2102 | journal_destroy(journal); | 2105 | journal_destroy(journal); |
2103 | return -EIO; | 2106 | return -EIO; |