diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2017-02-15 01:26:39 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-26 07:05:58 -0400 |
| commit | c2eeabe94145ea1c8b767162ede02cc8f55f4106 (patch) | |
| tree | d7969575a94d681e722319264b531e24ae0c4ec4 | |
| parent | dea2f1e0c5e9a325ae997b920cdc50bcefae804b (diff) | |
ext4: fix fencepost in s_first_meta_bg validation
commit 2ba3e6e8afc9b6188b471f27cf2b5e3cf34e7af2 upstream.
It is OK for s_first_meta_bg to be equal to the number of block group
descriptor blocks. (It rarely happens, but it shouldn't cause any
problems.)
https://bugzilla.kernel.org/show_bug.cgi?id=194567
Fixes: 3a4b77cd47bb837b8557595ec7425f281f2ca1fe
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Cc: Jiri Slaby <jslaby@suse.cz>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | fs/ext4/super.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext4/super.c b/fs/ext4/super.c index afe29ba42a4e..5fa9ba1de429 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c | |||
| @@ -3830,7 +3830,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent) | |||
| 3830 | db_count = (sbi->s_groups_count + EXT4_DESC_PER_BLOCK(sb) - 1) / | 3830 | db_count = (sbi->s_groups_count + EXT4_DESC_PER_BLOCK(sb) - 1) / |
| 3831 | EXT4_DESC_PER_BLOCK(sb); | 3831 | EXT4_DESC_PER_BLOCK(sb); |
| 3832 | if (ext4_has_feature_meta_bg(sb)) { | 3832 | if (ext4_has_feature_meta_bg(sb)) { |
| 3833 | if (le32_to_cpu(es->s_first_meta_bg) >= db_count) { | 3833 | if (le32_to_cpu(es->s_first_meta_bg) > db_count) { |
| 3834 | ext4_msg(sb, KERN_WARNING, | 3834 | ext4_msg(sb, KERN_WARNING, |
| 3835 | "first meta block group too large: %u " | 3835 | "first meta block group too large: %u " |
| 3836 | "(group descriptor block count %u)", | 3836 | "(group descriptor block count %u)", |