diff options
author | Nicolai Stange <nicstange@gmail.com> | 2016-05-05 22:43:04 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2016-05-05 22:43:04 -0400 |
commit | 816cd71b0c723a7296d14aadb8ff1ba42f6181d2 (patch) | |
tree | 661cecb56cc3ef7e89fe458459cd8dc9aaa09853 | |
parent | 466c3fb618b8520b75be37fcb115e9610663b945 (diff) |
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>
-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, | |||
912 | 912 | ||
913 | eh = ext_block_hdr(bh); | 913 | eh = ext_block_hdr(bh); |
914 | ppos++; | 914 | ppos++; |
915 | if (unlikely(ppos > depth)) { | ||
916 | put_bh(bh); | ||
917 | EXT4_ERROR_INODE(inode, | ||
918 | "ppos %d > depth %d", ppos, depth); | ||
919 | ret = -EFSCORRUPTED; | ||
920 | goto err; | ||
921 | } | ||
922 | path[ppos].p_bh = bh; | 915 | path[ppos].p_bh = bh; |
923 | path[ppos].p_hdr = eh; | 916 | path[ppos].p_hdr = eh; |
924 | } | 917 | } |