diff options
author | Nicolai Stange <nicstange@gmail.com> | 2016-05-06 05:43:04 +0300 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2016-05-06 05:43:04 +0300 |
commit | 816cd71b0c723a7296d14aadb8ff1ba42f6181d2 (patch) | |
tree | 661cecb56cc3ef7e89fe458459cd8dc9aaa09853 /fs/ext4/extents.c | |
parent | 466c3fb618b8520b75be37fcb115e9610663b945 (diff) | |
download | linux-816cd71b0c723a7296d14aadb8ff1ba42f6181d2.tar.xz |
ext4: remove unmeetable inconsisteny check from ext4_find_extent()
ext4_find_extent(), stripped down to the parts relevant to this patch,
reads as
ppos = 0;
i = depth;
while (i) {
--i;
++ppos;
if (unlikely(ppos > depth)) {
...
ret = -EFSCORRUPTED;
goto err;
}
}
Due to the loop's bounds, the condition ppos > depth can never be met.
Remove this dead code.
Signed-off-by: Nicolai Stange <nicstange@gmail.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/extents.c')
-rw-r--r-- | fs/ext4/extents.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index c53d5a8d2a79..2a2eef9c14e4 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -912,13 +912,6 @@ ext4_find_extent(struct inode *inode, ext4_lblk_t block, eh = ext_block_hdr(bh); ppos++; - if (unlikely(ppos > depth)) { - put_bh(bh); - EXT4_ERROR_INODE(inode, - "ppos %d > depth %d", ppos, depth); - ret = -EFSCORRUPTED; - goto err; - } path[ppos].p_bh = bh; path[ppos].p_hdr = eh; } |