Commit 6329592c authored by Filipe Manana's avatar Filipe Manana Committed by David Sterba
Browse files

btrfs: remove out label in scrub_find_fill_first_stripe() 



There is no point in having the label since all it does is return the
value in the 'ret' variable. Instead make every goto return directly
and remove the label.

Reviewed-by: default avatarJohannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: default avatarFilipe Manana <fdmanana@suse.com>
Reviewed-by: default avatarDavid Sterba <dsterba@suse.com>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent 55807025

fs/btrfs/scrub.c

+4 −4
Original line number Diff line number Diff line
@@ -1696,7 +1696,7 @@ static int scrub_find_fill_first_stripe(struct btrfs_block_group *bg, 
				     logical_len);

	/* Either error or not found. */

	if (ret)

		goto out;

		return ret;

	get_extent_info(extent_path, &extent_start, &extent_len, &extent_flags,

			&extent_gen);

	if (extent_flags & BTRFS_EXTENT_FLAG_TREE_BLOCK)
@@ -1729,7 +1729,7 @@ static int scrub_find_fill_first_stripe(struct btrfs_block_group *bg, 
		ret = find_first_extent_item(extent_root, extent_path, cur_logical,

					     stripe_end - cur_logical + 1);

		if (ret < 0)

			goto out;

			return ret;

		if (ret > 0) {

			ret = 0;

			break;
@@ -1763,7 +1763,7 @@ static int scrub_find_fill_first_stripe(struct btrfs_block_group *bg, 
						stripe->logical, stripe_end,

						stripe->csums, &csum_bitmap);

		if (ret < 0)

			goto out;

			return ret;

		if (ret > 0)

			ret = 0;
@@ -1773,7 +1773,7 @@ static int scrub_find_fill_first_stripe(struct btrfs_block_group *bg, 
		}

	}

	set_bit(SCRUB_STRIPE_FLAG_INITIALIZED, &stripe->state);

out:



	return ret;

}