ext4: add missing unlock in ext4_check_descriptors() on error path
authorLi Zefan <lizf@cn.fujitsu.com>
Mon, 8 Sep 2008 14:47:19 +0000 (10:47 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Mon, 8 Sep 2008 14:47:19 +0000 (10:47 -0400)
If there group descriptors are corrupted we need unlock the block
group lock before returning from the function; else we will oops when
freeing a spinlock which is still being held.

Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
fs/ext4/super.c

index 64e1c21eb5d223c8c4b2668a93969dcb132511e9..8175318abd84b2ca8440c3ec984ce5b4d7e4d22e 100644 (file)
@@ -1623,8 +1623,10 @@ static int ext4_check_descriptors(struct super_block *sb)
                               "Checksum for group %lu failed (%u!=%u)\n",
                               i, le16_to_cpu(ext4_group_desc_csum(sbi, i,
                               gdp)), le16_to_cpu(gdp->bg_checksum));
-                       if (!(sb->s_flags & MS_RDONLY))
+                       if (!(sb->s_flags & MS_RDONLY)) {
+                               spin_unlock(sb_bgl_lock(sbi, i));
                                return 0;
+                       }
                }
                spin_unlock(sb_bgl_lock(sbi, i));
                if (!flexbg_flag)