From f4befc129827632209779c71d804f6139ac03541 Mon Sep 17 00:00:00 2001 From: Arturo Borrero Gonzalez Date: Fri, 13 Sep 2013 14:05:51 +0200 Subject: src: xml: add parsing optional/mandatory flag Add an optional/mandatory flag to XML parsing. In some elements (ie regs), no flag is used because is always mandatory. DATA_NONE is created to indicate a non-parsed data_reg. Signed-off-by: Arturo Borrero Gonzalez Signed-off-by: Pablo Neira Ayuso --- src/set_elem.c | 37 ++++++++++++++++--------------------- 1 file changed, 16 insertions(+), 21 deletions(-) (limited to 'src/set_elem.c') diff --git a/src/set_elem.c b/src/set_elem.c index 2b2e414..9ad482b 100644 --- a/src/set_elem.c +++ b/src/set_elem.c @@ -378,41 +378,36 @@ EXPORT_SYMBOL(nft_set_elems_nlmsg_parse); #ifdef XML_PARSING int nft_mxml_set_elem_parse(mxml_node_t *tree, struct nft_set_elem *e) { - mxml_node_t *node; int set_elem_data; if (nft_mxml_num_parse(tree, "flags", MXML_DESCEND_FIRST, BASE_DEC, &e->set_elem_flags, - NFT_TYPE_U32) != 0) + NFT_TYPE_U32, NFT_XML_MAND) != 0) return -1; e->flags |= (1 << NFT_SET_ELEM_ATTR_FLAGS); - if (nft_mxml_data_reg_parse(tree, "key", &e->key) != DATA_VALUE) + if (nft_mxml_data_reg_parse(tree, "key", &e->key, + NFT_XML_MAND) != DATA_VALUE) return -1; e->flags |= (1 << NFT_SET_ELEM_ATTR_KEY); /* is not mandatory */ - node = mxmlFindElement(tree, tree, "data", NULL, NULL, - MXML_DESCEND_FIRST); - if (node != NULL && node->child != NULL) { - set_elem_data = nft_mxml_data_reg_parse(tree, "data", - &e->data); - switch (set_elem_data) { - case DATA_VALUE: - e->flags |= (1 << NFT_SET_ELEM_ATTR_DATA); - break; - case DATA_VERDICT: - e->flags |= (1 << NFT_SET_ELEM_ATTR_VERDICT); - break; - case DATA_CHAIN: - e->flags |= (1 << NFT_SET_ELEM_ATTR_CHAIN); - break; - default: - return -1; - } + set_elem_data = nft_mxml_data_reg_parse(tree, "data", + &e->data, NFT_XML_OPT); + switch (set_elem_data) { + case DATA_VALUE: + e->flags |= (1 << NFT_SET_ELEM_ATTR_DATA); + break; + case DATA_VERDICT: + e->flags |= (1 << NFT_SET_ELEM_ATTR_VERDICT); + break; + case DATA_CHAIN: + e->flags |= (1 << NFT_SET_ELEM_ATTR_CHAIN); + break; } + return 0; } #endif -- cgit v1.2.3