Re: [PATCH] lightnvm: pblk-gc: fix an error pointer dereference in init

[Matias Bjørling <m@xxxxxxxxxxx>]

On 04/13/2017 09:32 PM, Dan Carpenter wrote:
> These labels are reversed so we could end up dereferencing an error
> pointer or leaking.
>
> Fixes: 7f347ba6bb3a ("lightnvm: physical block device (pblk) target")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
>
> diff --git a/drivers/lightnvm/pblk-gc.c b/drivers/lightnvm/pblk-gc.c
> index 9b147cfd8a41..f173fd4ea947 100644
> --- a/drivers/lightnvm/pblk-gc.c
> +++ b/drivers/lightnvm/pblk-gc.c
> @@ -527,10 +527,10 @@ int pblk_gc_init(struct pblk *pblk)
>
>  	return 0;
>
> -fail_free_main_kthread:
> -	kthread_stop(gc->gc_ts);
>  fail_free_writer_kthread:
>  	kthread_stop(gc->gc_writer_ts);
> +fail_free_main_kthread:
> +	kthread_stop(gc->gc_ts);
>
>  	return ret;
>  }

Thanks Dan. I've applied it for 4.12.