summaryrefslogtreecommitdiffstats
path: root/fs/ext4
diff options
context:
space:
mode:
authorNicolai Stange <nicstange@gmail.com>2016-05-05 22:43:04 -0400
committerTheodore Ts'o <tytso@mit.edu>2016-05-05 22:43:04 -0400
commit816cd71b0c723a7296d14aadb8ff1ba42f6181d2 (patch)
tree661cecb56cc3ef7e89fe458459cd8dc9aaa09853 /fs/ext4
parent466c3fb618b8520b75be37fcb115e9610663b945 (diff)
downloadblackbird-op-linux-816cd71b0c723a7296d14aadb8ff1ba42f6181d2.tar.gz
blackbird-op-linux-816cd71b0c723a7296d14aadb8ff1ba42f6181d2.zip
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')
-rw-r--r--fs/ext4/extents.c7
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;
}
OpenPOWER on IntegriCloud