diff options
author | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-06-07 14:24:47 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-06-07 14:39:15 +0200 |
commit | e58b3f41aad8337749358c8f5393883ceab25dd9 (patch) | |
tree | f09306892abffbf1e410601519b9e610819401cf /src/rule.c | |
parent | dde20393f929f22636dcedd82e5a2fe81394139e (diff) |
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 <pablo@netfilter.org>
Diffstat (limited to 'src/rule.c')
-rw-r--r-- | src/rule.c | 6 |
1 files changed, 6 insertions, 0 deletions
@@ -88,6 +88,12 @@ void nft_rule_attr_unset(struct nft_rule *r, uint16_t attr) r->chain = NULL; } break; + case NFT_RULE_ATTR_HANDLE: + case NFT_RULE_ATTR_FLAGS: + case NFT_RULE_ATTR_COMPAT_PROTO: + case NFT_RULE_ATTR_COMPAT_FLAGS: + case NFT_RULE_ATTR_FAMILY: + break; default: return; } |