diff options
author | Chengguang Xu <cgxu519@zoho.com.cn> | 2019-07-23 07:21:54 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2019-07-31 06:04:42 -0400 |
commit | e5d395974e043cdcedcd84a0d41aaebb723786d8 (patch) | |
tree | 8ff04266c012c2ac4a3449149a9da23e3d489952 /fs/ext2/xattr.c | |
parent | 6fbacb8539a6659d446a9efabb538cfc007c1427 (diff) |
ext2: fix block range in ext2_data_block_valid()
For block validity we should check the block range
from start_block to start_block + count - 1, so fix
the range in ext2_data_block_valid() and also modify
the count argument properly in calling place.
Signed-off-by: Chengguang Xu <cgxu519@zoho.com.cn>
Link: https://lore.kernel.org/r/20190723112155.20329-1-cgxu519@zoho.com.cn
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ext2/xattr.c')
-rw-r--r-- | fs/ext2/xattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext2/xattr.c b/fs/ext2/xattr.c index 79369c13cc55..0456bc990b5e 100644 --- a/fs/ext2/xattr.c +++ b/fs/ext2/xattr.c | |||
@@ -794,7 +794,7 @@ ext2_xattr_delete_inode(struct inode *inode) | |||
794 | if (!EXT2_I(inode)->i_file_acl) | 794 | if (!EXT2_I(inode)->i_file_acl) |
795 | goto cleanup; | 795 | goto cleanup; |
796 | 796 | ||
797 | if (!ext2_data_block_valid(sbi, EXT2_I(inode)->i_file_acl, 0)) { | 797 | if (!ext2_data_block_valid(sbi, EXT2_I(inode)->i_file_acl, 1)) { |
798 | ext2_error(inode->i_sb, "ext2_xattr_delete_inode", | 798 | ext2_error(inode->i_sb, "ext2_xattr_delete_inode", |
799 | "inode %ld: xattr block %d is out of data blocks range", | 799 | "inode %ld: xattr block %d is out of data blocks range", |
800 | inode->i_ino, EXT2_I(inode)->i_file_acl); | 800 | inode->i_ino, EXT2_I(inode)->i_file_acl); |