diff options
author | Ana Rey <anarey@gmail.com> | 2014-06-19 17:07:07 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2014-06-23 19:50:17 +0200 |
commit | ee936a25d5a3eab9b94698a8f7c80742c6833003 (patch) | |
tree | eae9ee78e4afbb74b449f2f08cd6a31a6652bb6f /src/expr/byteorder.c | |
parent | ae90b05bb6d6910338c8a7a7b8904b7828251823 (diff) |
expr: byteorder: Use nft_rule_expr_set_* in the xml parsing code
Code refactoring to use nft_rule_expr_set_* in parse functions.
Signed-off-by: Ana Rey <anarey@gmail.com>
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, 14 insertions, 23 deletions
diff --git a/src/expr/byteorder.c b/src/expr/byteorder.c index e42d7b6..0b87eca 100644 --- a/src/expr/byteorder.c +++ b/src/expr/byteorder.c @@ -244,24 +244,19 @@ nft_rule_expr_byteorder_xml_parse(struct nft_rule_expr *e, mxml_node_t *tree, struct nft_parse_err *err) { #ifdef XML_PARSING - struct nft_expr_byteorder *byteorder = nft_expr_data(e); const char *op; int32_t ntoh; - uint32_t reg; + uint32_t sreg, dreg, len, size; - if (nft_mxml_reg_parse(tree, "sreg", ®, MXML_DESCEND_FIRST, - NFT_XML_MAND, err) != 0) + if (nft_mxml_reg_parse(tree, "sreg", &sreg, MXML_DESCEND_FIRST, + NFT_XML_MAND, err) < 0) return -1; + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_SREG, sreg); - byteorder->sreg = reg; - e->flags |= (1 << NFT_EXPR_BYTEORDER_SREG); - - if (nft_mxml_reg_parse(tree, "dreg", ®, MXML_DESCEND, NFT_XML_MAND, - err) != 0) + if (nft_mxml_reg_parse(tree, "dreg", &dreg, MXML_DESCEND, NFT_XML_MAND, + err) < 0) return -1; - - byteorder->dreg = reg; - e->flags |= (1 << NFT_EXPR_BYTEORDER_DREG); + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_DREG, dreg); op = nft_mxml_str_parse(tree, "op", MXML_DESCEND_FIRST, NFT_XML_MAND, err); @@ -271,23 +266,19 @@ nft_rule_expr_byteorder_xml_parse(struct nft_rule_expr *e, mxml_node_t *tree, ntoh = nft_str2ntoh(op); if (ntoh < 0) return -1; - - byteorder->op = ntoh; - e->flags |= (1 << NFT_EXPR_BYTEORDER_OP); + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_OP, ntoh); if (nft_mxml_num_parse(tree, "len", MXML_DESCEND_FIRST, BASE_DEC, - &byteorder->len, NFT_TYPE_U8, - NFT_XML_MAND, err) != 0) + &len, NFT_TYPE_U32, NFT_XML_MAND, + err) < 0) return -1; - - e->flags |= (1 << NFT_EXPR_BYTEORDER_LEN); + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_LEN, len); if (nft_mxml_num_parse(tree, "size", MXML_DESCEND_FIRST, BASE_DEC, - &byteorder->size, NFT_TYPE_U8, - NFT_XML_MAND, err) != 0) + &size, NFT_TYPE_U32, NFT_XML_MAND, + err) != 0) return -1; - - e->flags |= (1 << NFT_EXPR_BYTEORDER_SIZE); + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_SIZE, size); return 0; #else |