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>

diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c
index 50bdd82682fa39774249122e6a1e866449779494..edddd93d2118e477ca8570fe9282714da4e0ea6b 100644 (file)
--- 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 0054bde86aeaf55fd0146112db83d5eeaceb4561..370388fadf960a4d6fb075b402b172e9a82d400b 100644 (file)
--- 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;
 }