diff options
| author | Mohammad Shahid <mdshahid03@gmail.com> | 2026-07-03 23:03:01 +0530 |
|---|---|---|
| committer | Namjae Jeon <linkinjeon@kernel.org> | 2026-08-19 18:16:38 +0900 |
| commit | 5e150dc65d87bc533255408185dbdbc018058289 (patch) | |
| tree | f6f592fe231d6acdf7c53affebef7b8108007aae | |
| parent | e36dbad0882646443b347273111b50eeb66095ba (diff) | |
| download | linux-next-5e150dc65d87bc533255408185dbdbc018058289.tar.gz linux-next-5e150dc65d87bc533255408185dbdbc018058289.zip | |
ntfs: reparse: remove redundant NULL checks before kvfree()
kvfree() safely handles NULL pointers, so the explicit NULL checks
before calling kvfree() are unnecessary.
This issue was reported by ifnullfree.cocci.
Signed-off-by: Mohammad Shahid <mdshahid03@gmail.com>
Reviewed-by: Hyunchul Lee <hyc.lee@gmail.com>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
| -rw-r--r-- | fs/ntfs/reparse.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/ntfs/reparse.c b/fs/ntfs/reparse.c index fa523dc3691e..ced4c675d91f 100644 --- a/fs/ntfs/reparse.c +++ b/fs/ntfs/reparse.c @@ -317,8 +317,7 @@ unsigned int ntfs_make_symlink(struct ntfs_inode *ni) } else ni->flags &= ~FILE_ATTR_REPARSE_POINT; - if (reparse_attr) - kvfree(reparse_attr); + kvfree(reparse_attr); return mode; } @@ -358,8 +357,7 @@ unsigned int ntfs_reparse_tag_dt_types(struct ntfs_volume *vol, unsigned long mr } } - if (reparse_attr) - kvfree(reparse_attr); + kvfree(reparse_attr); iput(vi); return dt_type; |