Commit 6c675bd4 authored by Borislav Petkov's avatar Borislav Petkov Committed by Linus Torvalds

ext4: fix error handling in ext4_create_journal

Fix error handling in ext4_create_journal according to kernel conventions.
Signed-off-by: default avatarBorislav Petkov <bbpetkov@yahoo.de>
Cc: <linux-ext4@vger.kernel.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 952d9de1
...@@ -2158,6 +2158,7 @@ static int ext4_create_journal(struct super_block * sb, ...@@ -2158,6 +2158,7 @@ static int ext4_create_journal(struct super_block * sb,
unsigned int journal_inum) unsigned int journal_inum)
{ {
journal_t *journal; journal_t *journal;
int err;
if (sb->s_flags & MS_RDONLY) { if (sb->s_flags & MS_RDONLY) {
printk(KERN_ERR "EXT4-fs: readonly filesystem when trying to " printk(KERN_ERR "EXT4-fs: readonly filesystem when trying to "
...@@ -2165,13 +2166,15 @@ static int ext4_create_journal(struct super_block * sb, ...@@ -2165,13 +2166,15 @@ static int ext4_create_journal(struct super_block * sb,
return -EROFS; return -EROFS;
} }
if (!(journal = ext4_get_journal(sb, journal_inum))) journal = ext4_get_journal(sb, journal_inum);
if (!journal)
return -EINVAL; return -EINVAL;
printk(KERN_INFO "EXT4-fs: creating new journal on inode %u\n", printk(KERN_INFO "EXT4-fs: creating new journal on inode %u\n",
journal_inum); journal_inum);
if (jbd2_journal_create(journal)) { err = jbd2_journal_create(journal);
if (err) {
printk(KERN_ERR "EXT4-fs: error creating journal.\n"); printk(KERN_ERR "EXT4-fs: error creating journal.\n");
jbd2_journal_destroy(journal); jbd2_journal_destroy(journal);
return -EIO; return -EIO;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment