diff options
| author | Lv Ruyi <lv.ruyi@zte.com.cn> | 2022-04-11 03:22:52 +0000 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2022-05-16 17:03:12 +0200 |
| commit | 8aa1e49ea1fdcc585be4bcfe8c62684a087cc7c6 (patch) | |
| tree | 2002f72f5c12d51ed9c13a1cdf2cb626adf5ac40 /fs/btrfs | |
| parent | b027669449390ed549579f0a838c2de6a037dea6 (diff) | |
| download | linux-8aa1e49ea1fdcc585be4bcfe8c62684a087cc7c6.tar.bz2 | |
btrfs: remove unnecessary check of iput argument
iput() already handles NULL and non-NULL parameter, so it is not needed
to check that. This unifies all iput calls.
Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Lv Ruyi <lv.ruyi@zte.com.cn>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/relocation.c | 3 | ||||
| -rw-r--r-- | fs/btrfs/tree-log.c | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c index 50bdd82682fa..edddd93d2118 100644 --- a/fs/btrfs/relocation.c +++ b/fs/btrfs/relocation.c @@ -3846,8 +3846,7 @@ out: btrfs_end_transaction(trans); btrfs_btree_balance_dirty(fs_info); if (err) { - if (inode) - iput(inode); + iput(inode); inode = ERR_PTR(err); } return inode; diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index 0054bde86aea..370388fadf96 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c @@ -894,8 +894,7 @@ update_inode: btrfs_update_inode_bytes(BTRFS_I(inode), nbytes, drop_args.bytes_found); ret = btrfs_update_inode(trans, root, BTRFS_I(inode)); out: - if (inode) - iput(inode); + iput(inode); return ret; } |