aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBorislav Petkov <bbpetkov@yahoo.de>2007-07-16 02:41:45 -0400
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2007-07-16 12:05:51 -0400
commit952d9de116ad87261de106464a9eeec038c4cd14 (patch)
tree415354669cb3b2ccfe7ea066e0bcf79c0a4305de
parent1f1f642e2f092e37eb9038060eb0100c44f55a11 (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.c7
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;