Skip to content

Commit

Permalink
btrfs: send: cleanup unneeded return variable in changed_verity()
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
christian-heusel authored and kdave committed Oct 11, 2024
1 parent 66691c6 commit a0af493
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions fs/btrfs/send.c
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit a0af493

Please sign in to comment.