On Fri, Apr 09, 2021 at 03:15:05PM +0800, lixiaokeng wrote:
> If checker_cleanup_thread is called after cleanup_checkers,
> the checker_class will not be freed.
>
> Here, we use free_checker_class instead of checker_class_unref
> in checker_cleanup_thread.
>
> Signed-off-by: Lixiaokeng <lixiaokeng@xxxxxxxxxx>
> ---
> libmultipath/checkers.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libmultipath/checkers.c b/libmultipath/checkers.c
> index 2dd9915d..7cc80f0b 100644
> --- a/libmultipath/checkers.c
> +++ b/libmultipath/checkers.c
> @@ -368,7 +368,7 @@ static void checker_cleanup_thread(void *arg)
> {
> struct checker_class *cls = arg;
>
> - (void)checker_class_unref(cls);
> + (void)free_checker_class(cls);
> rcu_unregister_thread();
> }
>
> --
Reviewed-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/dm-devel
