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@xxxxxxxxxx> > Signed-off-by: Zhiqiang Liu <liuzhiqiang26@xxxxxxxxxx> > Signed-off-by: lixiaokeng <lixiaokeng@xxxxxxxxxx> > --- > 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 > -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel