diff options
author | Ana Rey <anarey@gmail.com> | 2014-06-19 17:07:08 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2014-06-23 19:51:42 +0200 |
commit | 199c2d597c6eb9a9f5603a1051f6c70c0a350a02 (patch) | |
tree | 323fa0cbada5924d2ea23a464d2a5a98e9c7fa5f /src/expr | |
parent | ee936a25d5a3eab9b94698a8f7c80742c6833003 (diff) |
expr: byteorder: Do not print unset values in xml file
It changes the parse and snprintf functions to omit unset values.
Signed-off-by: Ana Rey <anarey@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/expr')
-rw-r--r-- | src/expr/byteorder.c | 68 |
1 files changed, 39 insertions, 29 deletions
diff --git a/src/expr/byteorder.c b/src/expr/byteorder.c index 0b87eca..00e6b9c 100644 --- a/src/expr/byteorder.c +++ b/src/expr/byteorder.c @@ -249,36 +249,30 @@ nft_rule_expr_byteorder_xml_parse(struct nft_rule_expr *e, mxml_node_t *tree, uint32_t sreg, dreg, len, size; 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); + NFT_XML_MAND, err) == 0) + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_SREG, sreg); if (nft_mxml_reg_parse(tree, "dreg", &dreg, MXML_DESCEND, NFT_XML_MAND, - err) < 0) - return -1; - nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_DREG, dreg); + err) == 0) + 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); - if (op == NULL) - return -1; + if (op != NULL) { + ntoh = nft_str2ntoh(op); + if (ntoh < 0) + return -1; - ntoh = nft_str2ntoh(op); - if (ntoh < 0) - return -1; - nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_OP, ntoh); + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_OP, ntoh); + } if (nft_mxml_num_parse(tree, "len", MXML_DESCEND_FIRST, BASE_DEC, - &len, NFT_TYPE_U32, NFT_XML_MAND, - err) < 0) - return -1; - nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_LEN, len); + &len, NFT_TYPE_U32, NFT_XML_MAND, err) == 0) + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_LEN, len); if (nft_mxml_num_parse(tree, "size", MXML_DESCEND_FIRST, BASE_DEC, - &size, NFT_TYPE_U32, NFT_XML_MAND, - err) != 0) - return -1; - nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_SIZE, size); + &size, NFT_TYPE_U32, NFT_XML_MAND, err) == 0) + nft_rule_expr_set_u32(e, NFT_EXPR_BYTEORDER_SIZE, size); return 0; #else @@ -312,15 +306,31 @@ static int nft_rule_expr_byteorder_snprintf_xml(char *buf, size_t size, struct nft_expr_byteorder *byteorder = nft_expr_data(e); int len = size, offset = 0, ret; - ret = snprintf(buf, len, "<sreg>%u</sreg>" - "<dreg>%u</dreg>" - "<op>%s</op>" - "<len>%u</len>" - "<size>%u</size>", - byteorder->sreg, byteorder->dreg, - expr_byteorder_str[byteorder->op], - byteorder->len, byteorder->size); - SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + if (e->flags & (1 << NFT_EXPR_BYTEORDER_SREG)) { + ret = snprintf(buf + offset, len, "<sreg>%u</sreg>", + byteorder->sreg); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } + if (e->flags & (1 << NFT_EXPR_BYTEORDER_DREG)) { + ret = snprintf(buf + offset, len, "<dreg>%u</dreg>", + byteorder->dreg); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } + if (e->flags & (1 << NFT_EXPR_BYTEORDER_OP)) { + ret = snprintf(buf + offset, len, "<op>%s</op>", + expr_byteorder_str[byteorder->op]); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } + if (e->flags & (1 << NFT_EXPR_BYTEORDER_LEN)) { + ret = snprintf(buf + offset, len, "<len>%u</len>", + byteorder->len); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } + if (e->flags & (1 << NFT_EXPR_BYTEORDER_SIZE)) { + ret = snprintf(buf + offset, len, "<size>%u</size>", + byteorder->size); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } return offset; } |