|
 |
baeadd |
From 7b175a4fc9f81fe3f721d9a7ebb5b954a4fd7606 Mon Sep 17 00:00:00 2001
|
|
|
baeadd |
From: Liping Zhang <zlpnobody@gmail.com>
|
|
|
baeadd |
Date: Sun, 19 Mar 2017 22:01:10 +0800
|
|
|
baeadd |
Subject: [PATCH] examples: fix double free in nftc-helper-add
|
|
|
baeadd |
|
|
|
baeadd |
After inputting the following test command, core dump happened:
|
|
|
baeadd |
# ./examples/nfct-helper-add test 1
|
|
|
baeadd |
*** Error in
|
|
|
baeadd |
`.../libnetfilter_cthelper/examples/.libs/lt-nfct-helper-add':
|
|
|
baeadd |
double free or corruption (fasttop): 0x0000000001f3c070 ***
|
|
|
baeadd |
======= Backtrace: =========
|
|
|
baeadd |
/lib64/libc.so.6(+0x77de5)[0x7fd9ebe88de5]
|
|
|
baeadd |
/lib64/libc.so.6(+0x8022a)[0x7fd9ebe9122a]
|
|
|
baeadd |
/lib64/libc.so.6(cfree+0x4c)[0x7fd9ebe9478c]
|
|
|
baeadd |
[...]
|
|
|
baeadd |
|
|
|
baeadd |
Because "struct nfct_helper_policy *p" had been freed by nfct_helper_free,
|
|
|
baeadd |
so there's no need to invoke nfct_helper_policy_free again, otherwise
|
|
|
baeadd |
double free error will happen.
|
|
|
baeadd |
|
|
|
baeadd |
Signed-off-by: Liping Zhang <zlpnobody@gmail.com>
|
|
|
baeadd |
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
|
|
|
baeadd |
(cherry picked from commit 9f223e612d3b0be6e4dca84e1db8042dbec64e93)
|
|
|
baeadd |
---
|
|
|
baeadd |
examples/nfct-helper-add.c | 2 +-
|
|
|
baeadd |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
baeadd |
|
|
|
baeadd |
diff --git a/examples/nfct-helper-add.c b/examples/nfct-helper-add.c
|
|
|
baeadd |
index 6c476261b75e8..cb7291e8ec4b0 100644
|
|
|
baeadd |
--- a/examples/nfct-helper-add.c
|
|
|
baeadd |
+++ b/examples/nfct-helper-add.c
|
|
|
baeadd |
@@ -32,6 +32,7 @@ int main(int argc, char *argv[])
|
|
|
baeadd |
nfct_helper_attr_set_u16(nfct_helper, NFCTH_ATTR_PROTO_L3NUM, AF_INET);
|
|
|
baeadd |
nfct_helper_attr_set_u8(nfct_helper, NFCTH_ATTR_PROTO_L4NUM, IPPROTO_TCP);
|
|
|
baeadd |
|
|
|
baeadd |
+ /* Will be freed by nfct_helper_free. */
|
|
|
baeadd |
p = nfct_helper_policy_alloc();
|
|
|
baeadd |
if (p == NULL) {
|
|
|
baeadd |
perror("OOM");
|
|
|
baeadd |
@@ -49,7 +50,6 @@ int main(int argc, char *argv[])
|
|
|
baeadd |
nfct_helper_nlmsg_build_payload(nlh, nfct_helper);
|
|
|
baeadd |
|
|
|
baeadd |
nfct_helper_free(nfct_helper);
|
|
|
baeadd |
- nfct_helper_policy_free(p);
|
|
|
baeadd |
|
|
|
baeadd |
nl = mnl_socket_open(NETLINK_NETFILTER);
|
|
|
baeadd |
if (nl == NULL) {
|
|
|
baeadd |
--
|
|
|
baeadd |
2.21.0
|
|
|
baeadd |
|