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/set.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/set.c')
-rw-r--r-- | src/set.c | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -435,10 +435,12 @@ int nft_set_nlmsg_parse(const struct nlmsghdr *nlh, struct nft_set *s) return -1; if (tb[NFTA_SET_TABLE]) { + xfree(s->table); s->table = strdup(mnl_attr_get_str(tb[NFTA_SET_TABLE])); s->flags |= (1 << NFT_SET_ATTR_TABLE); } if (tb[NFTA_SET_NAME]) { + xfree(s->name); s->name = strdup(mnl_attr_get_str(tb[NFTA_SET_NAME])); s->flags |= (1 << NFT_SET_ATTR_NAME); } |