Discussion:
[PATCH] elf_info.c: fix memory leak in get_kcore_dump_loads()
Pingfan Liu
2017-03-21 03:28:19 UTC
Permalink
Signed-off-by: Pingfan Liu <***@redhat.com>
---
elf_info.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/elf_info.c b/elf_info.c
index d84553a..35e754b 100644
--- a/elf_info.c
+++ b/elf_info.c
@@ -893,12 +893,14 @@ int get_kcore_dump_loads(void)
|| !is_phys_addr(p->virt_start))
continue;
if (j >= loads)
+ free(pls)
return FALSE;

if (j == 0) {
offset_pt_load_memory = p->file_offset;
if (offset_pt_load_memory == 0) {
ERRMSG("Can't get the offset of page data.\n");
+ free(pls)
return FALSE;
}
}
--
2.7.4
Atsushi Kumagai
2017-03-22 06:59:29 UTC
Permalink
Hello Pingfan,

It would be helpful if you could mention that the patch is for
makedumpfile when you post that, otherwise I could overlook that.

Anyway, I'll merge this patch into v1.6.2 after adding semicolon
to the back of "free(pls)".

Thanks,
Atsushi Kumagai
Post by Pingfan Liu
---
elf_info.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/elf_info.c b/elf_info.c
index d84553a..35e754b 100644
--- a/elf_info.c
+++ b/elf_info.c
@@ -893,12 +893,14 @@ int get_kcore_dump_loads(void)
|| !is_phys_addr(p->virt_start))
continue;
if (j >= loads)
+ free(pls)
return FALSE;
if (j == 0) {
offset_pt_load_memory = p->file_offset;
if (offset_pt_load_memory == 0) {
ERRMSG("Can't get the offset of page data.\n");
+ free(pls)
return FALSE;
}
}
--
2.7.4
_______________________________________________
kexec mailing list
http://lists.infradead.org/mailman/listinfo/kexec
Pingfan Liu
2017-03-22 07:18:09 UTC
Permalink
----- Original Message -----
Sent: Wednesday, March 22, 2017 2:59:29 PM
Subject: RE: [PATCH] elf_info.c: fix memory leak in get_kcore_dump_loads()
Hello Pingfan,
It would be helpful if you could mention that the patch is for
makedumpfile when you post that, otherwise I could overlook that.
Sorry, and I will add such info.

Regards,
Pingfan
Anyway, I'll merge this patch into v1.6.2 after adding semicolon
to the back of "free(pls)".
Thanks,
Atsushi Kumagai
Post by Pingfan Liu
---
elf_info.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/elf_info.c b/elf_info.c
index d84553a..35e754b 100644
--- a/elf_info.c
+++ b/elf_info.c
@@ -893,12 +893,14 @@ int get_kcore_dump_loads(void)
|| !is_phys_addr(p->virt_start))
continue;
if (j >= loads)
+ free(pls)
return FALSE;
if (j == 0) {
offset_pt_load_memory = p->file_offset;
if (offset_pt_load_memory == 0) {
ERRMSG("Can't get the offset of page data.\n");
+ free(pls)
return FALSE;
}
}
--
2.7.4
_______________________________________________
kexec mailing list
http://lists.infradead.org/mailman/listinfo/kexec
_______________________________________________
kexec mailing list
http://lists.infradead.org/mailman/listinfo/kexec
Loading...