diff options
| author | Carlos Falgueras García <carlosfg@riseup.net> | 2015-08-05 12:23:34 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2015-08-18 01:52:01 +0200 |
| commit | 6e188637782c6c62c7bcade808e59d17639d677d (patch) | |
| tree | 658c9400a8f7beb07b00b445b9a7d0eba6bcea89 /src/chain.c | |
| parent | 0edeb667a2cfdd36d8c89de88e1a446595ce95d8 (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/chain.c')
| -rw-r--r-- | src/chain.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/chain.c b/src/chain.c index ed9d9e4..4f9c6b7 100644 --- a/src/chain.c +++ b/src/chain.c @@ -534,6 +534,7 @@ int nft_chain_nlmsg_parse(const struct nlmsghdr *nlh, struct nft_chain *c) c->flags |= (1 << NFT_CHAIN_ATTR_NAME); } if (tb[NFTA_CHAIN_TABLE]) { + xfree(c->table); c->table = strdup(mnl_attr_get_str(tb[NFTA_CHAIN_TABLE])); c->flags |= (1 << NFT_CHAIN_ATTR_TABLE); } @@ -560,6 +561,7 @@ int nft_chain_nlmsg_parse(const struct nlmsghdr *nlh, struct nft_chain *c) c->flags |= (1 << NFT_CHAIN_ATTR_HANDLE); } if (tb[NFTA_CHAIN_TYPE]) { + xfree(c->type); c->type = strdup(mnl_attr_get_str(tb[NFTA_CHAIN_TYPE])); c->flags |= (1 << NFT_CHAIN_ATTR_TYPE); } |