btrfs: send: cleanup unneeded return variable in changed_verity()
authorChristian Heusel <christian@heusel.eu>
Thu, 10 Oct 2024 13:25:25 +0000 (15:25 +0200)
committerDavid Sterba <dsterba@suse.com>
Fri, 11 Oct 2024 17:54:58 +0000 (19:54 +0200)
As all changed_* functions need to return something, just return 0
directly here, as the verity status is passed via the context.

Reported by LKP: fs/btrfs/send.c:6877:5-8: Unneeded variable: "ret". Return "0" on line 6883

Reported-by: kernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/oe-kbuild-all/202410092305.WbyqspH8-lkp@intel.com/
Signed-off-by: Christian Heusel <christian@heusel.eu>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/send.c

index 27306d9..b068469 100644 (file)
@@ -7190,13 +7190,11 @@ static int changed_extent(struct send_ctx *sctx,
 
 static int changed_verity(struct send_ctx *sctx, enum btrfs_compare_tree_result result)
 {
-       int ret = 0;
-
        if (!sctx->cur_inode_new_gen && !sctx->cur_inode_deleted) {
                if (result == BTRFS_COMPARE_TREE_NEW)
                        sctx->cur_inode_needs_verity = true;
        }
-       return ret;
+       return 0;
 }
 
 static int dir_changed(struct send_ctx *sctx, u64 dir)