From bc04bd902c7c2971154a5b97d494fc3a0c600a74 Mon Sep 17 00:00:00 2001 From: Ana Rey Date: Thu, 19 Jun 2014 19:34:24 +0200 Subject: expr: cmp: Do not print unset values in json file It changes the parse and snprintf functions to omit unset values. Signed-off-by: Ana Rey Signed-off-by: Pablo Neira Ayuso --- src/expr/cmp.c | 41 +++++++++++++++++++++-------------------- 1 file changed, 21 insertions(+), 20 deletions(-) (limited to 'src/expr/cmp.c') diff --git a/src/expr/cmp.c b/src/expr/cmp.c index 2ed05f5..8b0e54d 100644 --- a/src/expr/cmp.c +++ b/src/expr/cmp.c @@ -183,26 +183,22 @@ static int nft_rule_expr_cmp_json_parse(struct nft_rule_expr *e, json_t *root, uint32_t uval32; int base; - if (nft_jansson_parse_val(root, "sreg", NFT_TYPE_U32, &uval32, err) < 0) - return -1; - - nft_rule_expr_set_u32(e, NFT_EXPR_CMP_SREG, uval32); + if (nft_jansson_parse_val(root, "sreg", NFT_TYPE_U32, &uval32, + err) == 0) + nft_rule_expr_set_u32(e, NFT_EXPR_CMP_SREG, uval32); op = nft_jansson_parse_str(root, "op", err); - if (op == NULL) - return -1; - - base = nft_str2cmp(op); - if (base < 0) - return -1; + if (op != NULL) { + base = nft_str2cmp(op); + if (base < 0) + return -1; - nft_rule_expr_set_u32(e, NFT_EXPR_CMP_OP, base); + nft_rule_expr_set_u32(e, NFT_EXPR_CMP_OP, base); + } if (nft_jansson_data_reg_parse(root, "cmpdata", - &cmp->data, err) != DATA_VALUE) - return -1; - - e->flags |= (1 << NFT_EXPR_CMP_DATA); + &cmp->data, err) == DATA_VALUE) + e->flags |= (1 << NFT_EXPR_CMP_DATA); return 0; #else @@ -252,11 +248,16 @@ static int nft_rule_expr_cmp_snprintf_json(char *buf, size_t size, struct nft_expr_cmp *cmp = nft_expr_data(e); int len = size, offset = 0, ret; - ret = snprintf(buf, len, "\"sreg\":%u,\"op\":\"%s\",", - cmp->sreg, expr_cmp_str[cmp->op]); - SNPRINTF_BUFFER_SIZE(ret, size, len, offset); - - ret = nft_data_reg_snprintf(buf+offset, len, &cmp->data, + if (e->flags & (1 << NFT_EXPR_CMP_SREG)) { + ret = snprintf(buf + offset, len, "\"sreg\":%u,", cmp->sreg); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } + if (e->flags & (1 << NFT_EXPR_CMP_OP)) { + ret = snprintf(buf + offset, len, "\"op\":\"%s\",", + expr_cmp_str[cmp->op]); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } + ret = nft_data_reg_snprintf(buf + offset, len, &cmp->data, NFT_OUTPUT_JSON, 0, DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, size, len, offset); -- cgit v1.2.3