[dm-devel] [PATCH] libmultipath: fix a memory leak in disassemble_status func
Benjamin Marzinski
bmarzins at redhat.com
Tue Jul 28 16:20:00 UTC 2020
On Tue, Jul 28, 2020 at 09:48:06PM +0800, Zhiqiang Liu wrote:
>
> In disassemble_status func, for dealing with selector args,
> word is allocated by get_word func. However, word is not freed.
> Then a memory leak occurs.
>
> Here, we call FREE(word) to free word.
Err... At the risk of sounding stupid, There is a least-pending
selector? Hannes, you added this code (commit 35ad40b4 "leastpending IO
loadbalancing is not displayed properly"). Was this a Suse thing? Is it
still a Suse thing, or have queue-length and service-time replaced it?
At any rate, for the patch itself (assuming that the correct answer
isn't to just delete the least-pending code),
Reviewed-by: Benjamin Marzinski <bmarzins at redhat.com>
> Signed-off-by: Zhiqiang Liu <liuzhiqiang26 at huawei.com>
> Signed-off-by: lixiaokeng <lixiaokeng at huawei.com>
> ---
> libmultipath/dmparser.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/libmultipath/dmparser.c b/libmultipath/dmparser.c
> index 3dc77242..a4a989b2 100644
> --- a/libmultipath/dmparser.c
> +++ b/libmultipath/dmparser.c
> @@ -584,6 +584,7 @@ int disassemble_status(char *params, struct multipath *mpp)
> &def_minio) == 1 &&
> def_minio != mpp->minio)
> mpp->minio = def_minio;
> + FREE(word);
> } else
> p += get_word(p, NULL);
> }
> --
> 2.24.0.windows.2
>
More information about the dm-devel
mailing list