diff options
author | luo penghao <luo.penghao@zte.com.cn> | 2021-11-04 07:40:07 +0100 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2022-01-10 19:25:56 +0100 |
commit | fac888b2be9993d0c740013d26d69c8784acc293 (patch) | |
tree | de9e0e85f1f940925d8d40de0af3a17cb15275a8 /fs/ext4 | |
parent | ext4: remove redundant statement (diff) | |
download | linux-fac888b2be9993d0c740013d26d69c8784acc293.tar.xz linux-fac888b2be9993d0c740013d26d69c8784acc293.zip |
ext4: remove unused assignments
The eh assignment in these two places is meaningless, because the
function will goto to merge, which will not use eh.
The clang_analyzer complains as follows:
fs/ext4/extents.c:1988:4 warning:
fs/ext4/extents.c:2016:4 warning:
Value stored to 'eh' 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/20211104064007.2919-1-luo.penghao@zte.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/ext4')
-rw-r--r-- | fs/ext4/extents.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index c3e76a5de661..299d27d04b2b 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -2025,7 +2025,6 @@ int ext4_ext_insert_extent(handle_t *handle, struct inode *inode, + ext4_ext_get_actual_len(newext)); if (unwritten) ext4_ext_mark_unwritten(ex); - eh = path[depth].p_hdr; nearex = ex; goto merge; } @@ -2054,7 +2053,6 @@ prepend: + ext4_ext_get_actual_len(newext)); if (unwritten) ext4_ext_mark_unwritten(ex); - eh = path[depth].p_hdr; nearex = ex; goto merge; } |