diff options
author | Ana Rey <anarey@gmail.com> | 2014-06-23 11:38:20 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2014-06-23 19:48:53 +0200 |
commit | 752f2afcb75780940cbabe4c3a95efddfc6fcec0 (patch) | |
tree | f421cda75ecce4e7409bdb32fd0dbf7baeaa5e68 /src | |
parent | fe61f81e54dc3647adfedc6659dd6617078c84f4 (diff) |
expr: target: Do not print unset values in json file
It changes the parse and snrintf 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')
-rw-r--r-- | src/expr/target.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/expr/target.c b/src/expr/target.c index 101f3d0..7ebeda6 100644 --- a/src/expr/target.c +++ b/src/expr/target.c @@ -178,10 +178,8 @@ nft_rule_expr_target_json_parse(struct nft_rule_expr *e, json_t *root, const char *name; name = nft_jansson_parse_str(root, "name", err); - if (name == NULL) - return -1; - - nft_rule_expr_set_str(e, NFT_EXPR_TG_NAME, name); + if (name != NULL) + nft_rule_expr_set_str(e, NFT_EXPR_TG_NAME, name); return 0; #else @@ -217,8 +215,10 @@ static int nft_rule_exp_target_snprintf_json(char *buf, size_t len, struct nft_expr_target *target = nft_expr_data(e); int ret, size = len, offset = 0; - ret = snprintf(buf, len, "\"name\":\"%s\"", target->name); - SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + if (e->flags & (1 << NFT_EXPR_TG_NAME)) { + ret = snprintf(buf, len, "\"name\":\"%s\"", target->name); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + } return offset; } |