diff options
| author | luo penghao <luo.penghao@zte.com.cn> | 2021-12-08 08:51:57 +0100 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2022-01-10 19:25:56 +0100 |
| commit | a6dbc76c4d9cc961bfb31a495fb27ee06e037e35 (patch) | |
| tree | e4eb728f09c72a2877f620b711116bb06ef6f17a /fs/ext4/move_extent.c | |
| parent | ext4: drop an always true check (diff) | |
| download | linux-a6dbc76c4d9cc961bfb31a495fb27ee06e037e35.tar.xz linux-a6dbc76c4d9cc961bfb31a495fb27ee06e037e35.zip | |
ext4: remove redundant o_start statement
The if will goto out of the loop, and until the end of the
function execution, o_start will not be used again.
The clang_analyzer complains as follows:
fs/ext4/move_extent.c:635:5 warning:
Value stored to 'o_start' is never read
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: luo penghao <luo.penghao@zte.com.cn>
Link: https://lore.kernel.org/r/20211208075157.404535-1-luo.penghao@zte.com.cn
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/move_extent.c')
| -rw-r--r-- | fs/ext4/move_extent.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index 64a579734f93..95aa212f0863 100644 --- a/fs/ext4/move_extent.c +++ b/fs/ext4/move_extent.c @@ -632,7 +632,6 @@ ext4_move_extents(struct file *o_filp, struct file *d_filp, __u64 orig_blk, /* Check hole before the start pos */ if (cur_blk + cur_len - 1 < o_start) { if (next_blk == EXT_MAX_BLOCKS) { - o_start = o_end; ret = -ENODATA; goto out; } |