On 26.04.2018 22:23, je...@suse.com wrote:
> From: Jeff Mahoney <je...@suse.com>
>
> If we fail to allocate memory for a path, don't bother trying to
> insert the qgroup status item. We haven't done anything yet and it'll
> fail also. Just print an error and be done with it.
>
> Signed-off-by: Jeff Mahoney <je...@suse.com>
nit: So the code is correct however, having the out label there is
really ugly. What about on path alloc failure just have the print in the
if branch do goto done?
Reviewed-by: Nikolay Borisov <nbori...@suse.com>
> ---
> fs/btrfs/qgroup.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/btrfs/qgroup.c b/fs/btrfs/qgroup.c
> index 8de423a0c7e3..4c0978bce5b9 100644
> --- a/fs/btrfs/qgroup.c
> +++ b/fs/btrfs/qgroup.c
> @@ -2648,7 +2648,6 @@ static void btrfs_qgroup_rescan_worker(struct
> btrfs_work *work)
> btrfs_end_transaction(trans);
> }
>
> -out:
> btrfs_free_path(path);
>
> mutex_lock(&fs_info->qgroup_rescan_lock);
> @@ -2688,6 +2687,7 @@ static void btrfs_qgroup_rescan_worker(struct
> btrfs_work *work)
> btrfs_info(fs_info, "qgroup scan completed%s",
> err > 0 ? " (inconsistency flag cleared)" : "");
> } else {
> +out:
> btrfs_err(fs_info, "qgroup scan failed with %d", err);
> }
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
