diff options
| author | Namjae Jeon <linkinjeon@kernel.org> | 2026-05-21 21:30:01 +0900 |
|---|---|---|
| committer | Namjae Jeon <linkinjeon@kernel.org> | 2026-05-25 11:41:21 +0900 |
| commit | b2dad4fe33c53da811d00dcaa9368bf07a62f434 (patch) | |
| tree | 22b5b12d16826f7c55486c6e20093784312b0b5f /fs | |
| parent | 467f26c0f9f13ed3fdc66ecd2f17b31bf411a556 (diff) | |
| download | linux-next-history-b2dad4fe33c53da811d00dcaa9368bf07a62f434.tar.gz | |
ntfs: remove unnecessary NULL checks before kfree
NULL check before kfree() is unnecessary and triggers coccinelle warnings.
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/ntfs/logfile.c | 3 | ||||
| -rw-r--r-- | fs/ntfs/super.c | 5 |
2 files changed, 3 insertions, 5 deletions
diff --git a/fs/ntfs/logfile.c b/fs/ntfs/logfile.c index d3f25d8e29f9d..9df8c3095ca48 100644 --- a/fs/ntfs/logfile.c +++ b/fs/ntfs/logfile.c @@ -622,8 +622,7 @@ is_empty: ntfs_debug("Done."); return true; err_out: - if (rstr1_ph) - kvfree(rstr1_ph); + kvfree(rstr1_ph); return false; } diff --git a/fs/ntfs/super.c b/fs/ntfs/super.c index 312acb41c2ed5..2a5ad7d56bc26 100644 --- a/fs/ntfs/super.c +++ b/fs/ntfs/super.c @@ -1634,7 +1634,7 @@ static void ntfs_volume_free(struct ntfs_volume *vol) vol->upcase = NULL; } - if (!ntfs_nr_upcase_users && default_upcase) { + if (!ntfs_nr_upcase_users) { kvfree(default_upcase); default_upcase = NULL; } @@ -1649,8 +1649,7 @@ static void ntfs_volume_free(struct ntfs_volume *vol) unload_nls(vol->nls_map); - if (vol->lcn_empty_bits_per_page) - kvfree(vol->lcn_empty_bits_per_page); + kvfree(vol->lcn_empty_bits_per_page); kfree(vol->volume_label); kfree(vol); } |