diff options
author | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-07-18 20:01:27 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-07-18 20:08:25 +0200 |
commit | 6cdc49ffbe0a672c27f4fffe831563b12a8e2e61 (patch) | |
tree | 76e1b5318bfdd9627f8ab5138293cbe569c25da4 /src/expr/byteorder.c | |
parent | 5ecac519cd78043d0a5bfead1922a683d32db9d2 (diff) |
src: simplify getter logic
This patch refactors the getter code to simplify it. The default
cases have been removed so gcc will spot a warning if an attribute
is not handled appropriately.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/expr/byteorder.c')
-rw-r--r-- | src/expr/byteorder.c | 37 |
1 files changed, 10 insertions, 27 deletions
diff --git a/src/expr/byteorder.c b/src/expr/byteorder.c index 2326961..e3b9a5e 100644 --- a/src/expr/byteorder.c +++ b/src/expr/byteorder.c @@ -67,37 +67,20 @@ nft_rule_expr_byteorder_get(const struct nft_rule_expr *e, uint16_t type, switch(type) { case NFT_EXPR_BYTEORDER_SREG: - if (e->flags & (1 << NFT_EXPR_BYTEORDER_SREG)) { - *data_len = sizeof(byteorder->sreg); - return &byteorder->sreg; - } - break; + *data_len = sizeof(byteorder->sreg); + return &byteorder->sreg; case NFT_EXPR_BYTEORDER_DREG: - if (e->flags & (1 << NFT_EXPR_BYTEORDER_DREG)) { - *data_len = sizeof(byteorder->dreg); - return &byteorder->dreg; - } - break; + *data_len = sizeof(byteorder->dreg); + return &byteorder->dreg; case NFT_EXPR_BYTEORDER_OP: - if (e->flags & (1 << NFT_EXPR_BYTEORDER_OP)) { - *data_len = sizeof(byteorder->op); - return &byteorder->op; - } - break; + *data_len = sizeof(byteorder->op); + return &byteorder->op; case NFT_EXPR_BYTEORDER_LEN: - if (e->flags & (1 << NFT_EXPR_BYTEORDER_LEN)) { - *data_len = sizeof(byteorder->len); - return &byteorder->len; - } - break; + *data_len = sizeof(byteorder->len); + return &byteorder->len; case NFT_EXPR_BYTEORDER_SIZE: - if (e->flags & (1 << NFT_EXPR_BYTEORDER_SIZE)) { - *data_len = sizeof(byteorder->size); - return &byteorder->size; - } - break; - default: - break; + *data_len = sizeof(byteorder->size); + return &byteorder->size; } return NULL; } |