diff options
author | Zeng Jingxiang <linuszeng@tencent.com> | 2022-07-20 16:30:29 +0800 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2022-07-26 12:46:36 +0200 |
commit | bd6e21a9044ffc0d15cc362dcc10662e644a43bc (patch) | |
tree | 3065c978c334209e6a8bd443f5b77ebe51c90aec /fs/ext2 | |
parent | 9888725d8a640d67abe6139e84ff41bafe89b11e (diff) |
fs/reiserfs/inode: remove dead code in _get_block_create_0()
Since commit 27b3a5c51b50 ("kill-the-bkl/reiserfs: drop the fs race
watchdog from _get_block_create_0()"), which removed a label that may
have the pointer 'p' touched in its control flow, related if statements
now eval to constant value now. Just remove them.
Assigning value NULL to p here
293 char *p = NULL;
In the following conditional expression, the value of p is always NULL,
As a result, the kunmap() cannot be executed.
308 if (p)
309 kunmap(bh_result->b_page);
355 if (p)
356 kunmap(bh_result->b_page);
366 if (p)
367 kunmap(bh_result->b_page);
Also, the kmap() cannot be executed.
399 if (!p)
400 p = (char *)kmap(bh_result->b_page);
[JK: Removed unnecessary initialization of 'p' to NULL]
Signed-off-by: Zeng Jingxiang <linuszeng@tencent.com>
Signed-off-by: Kairui Song <kasong@tencent.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Link: https://lore.kernel.org/r/20220720083029.1065578-1-zengjx95@gmail.com
Diffstat (limited to 'fs/ext2')
0 files changed, 0 insertions, 0 deletions