summaryrefslogtreecommitdiffstats
path: root/src/rule.c
diff options
context:
space:
mode:
authorCarlos Falgueras García <carlosfg@riseup.net>2015-08-05 12:23:34 +0200
committerPablo Neira Ayuso <pablo@netfilter.org>2015-08-18 01:52:01 +0200
commit6e188637782c6c62c7bcade808e59d17639d677d (patch)
tree658c9400a8f7beb07b00b445b9a7d0eba6bcea89 /src/rule.c
parent0edeb667a2cfdd36d8c89de88e1a446595ce95d8 (diff)
src: fix memory leaks at nft_[object]_nlmsg_parse
Free object attributes before overwrite it. Fix 'nlmsg_parse' methods of following objects: 'table', 'chain', 'rule', 'set' and 'set_element'. Signed-off-by: Carlos Falgueras García <carlosfg@riseup.net> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/rule.c')
-rw-r--r--src/rule.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/rule.c b/src/rule.c
index 04088ed..ebaa3ed 100644
--- a/src/rule.c
+++ b/src/rule.c
@@ -425,10 +425,12 @@ int nft_rule_nlmsg_parse(const struct nlmsghdr *nlh, struct nft_rule *r)
return -1;
if (tb[NFTA_RULE_TABLE]) {
+ xfree(r->table);
r->table = strdup(mnl_attr_get_str(tb[NFTA_RULE_TABLE]));
r->flags |= (1 << NFT_RULE_ATTR_TABLE);
}
if (tb[NFTA_RULE_CHAIN]) {
+ xfree(r->chain);
r->chain = strdup(mnl_attr_get_str(tb[NFTA_RULE_CHAIN]));
r->flags |= (1 << NFT_RULE_ATTR_CHAIN);
}