[dm-devel] [PATCH] libmultipath: fix a memory leak in disassemble_status func

Zhiqiang Liu liuzhiqiang26 at huawei.com
Tue Jul 28 13:48:06 UTC 2020


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.

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