From e58b3f41aad8337749358c8f5393883ceab25dd9 Mon Sep 17 00:00:00 2001 From: Pablo Neira Ayuso Date: Fri, 7 Jun 2013 14:24:47 +0200 Subject: src: fix nft_*_unset function attribute that don't release data In (dde2039 src: add nft_*_unset functions), I mangled Arturo's patch to add a default case, but he was intentionally not adding it to unset attributes that require no memory releases. I prefered to add the attributes explicitly in the switch rather on failing back on the default action. Signed-off-by: Pablo Neira Ayuso --- src/chain.c | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'src/chain.c') diff --git a/src/chain.c b/src/chain.c index 530e045..7775dfc 100644 --- a/src/chain.c +++ b/src/chain.c @@ -79,6 +79,15 @@ void nft_chain_attr_unset(struct nft_chain *c, uint16_t attr) c->type = NULL; } break; + case NFT_CHAIN_ATTR_NAME: + case NFT_CHAIN_ATTR_HOOKNUM: + case NFT_CHAIN_ATTR_PRIO: + case NFT_CHAIN_ATTR_POLICY: + case NFT_CHAIN_ATTR_BYTES: + case NFT_CHAIN_ATTR_PACKETS: + case NFT_CHAIN_ATTR_HANDLE: + case NFT_CHAIN_ATTR_FAMILY: + break; default: return; } -- cgit v1.2.3