adfbb8ebd5
If mdcheck_start service can finish check action, it doesn't need to start mdcheck_continue service. So in rhel only patch mdcheck.patch, we stop mdcheck_continue timer. And there is a history problem. It needed KillMode=none before, so it removed the upstream patch 52c67fcdd. Now this problem has been fixed, so we can do the backport more easilly now. We don't need to remove the upstream patch here again. Resolves: rhbz#2116418, rhbz#2150862, rhbz#2159584 Signed-off-by: Xiao Ni <xni@redhat.com>
33 lines
842 B
Diff
33 lines
842 B
Diff
From 06ef619582b47af89eb094c164fc5effd46d6048 Mon Sep 17 00:00:00 2001
|
|
From: Wu Guanghao <wuguanghao3@huawei.com>
|
|
Date: Sat, 4 Mar 2023 00:21:31 +0800
|
|
Subject: [PATCH 094/125] util.c: fix memleak in parse_layout_faulty()
|
|
|
|
char *m is allocated by xstrdup but not free() before return, will cause
|
|
a memory leak
|
|
|
|
Signed-off-by: Wu Guanghao <wuguanghao3@huawei.com>
|
|
Acked-by: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
|
|
Acked-by: Coly Li <colyli@suse.de>
|
|
Signed-off-by: Jes Sorensen <jes@trained-monkey.org>
|
|
---
|
|
util.c | 2 ++
|
|
1 file changed, 2 insertions(+)
|
|
|
|
diff --git a/util.c b/util.c
|
|
index b0b7aec4..9f1e1f7c 100644
|
|
--- a/util.c
|
|
+++ b/util.c
|
|
@@ -432,6 +432,8 @@ int parse_layout_faulty(char *layout)
|
|
m = xstrdup(layout);
|
|
m[ln] = 0;
|
|
mode = map_name(faultylayout, m);
|
|
+ free(m);
|
|
+
|
|
if (mode == UnSet)
|
|
return -1;
|
|
|
|
--
|
|
2.38.1
|
|
|