diff options
author | Phil Sutter <phil@nwl.cc> | 2018-08-29 16:23:26 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2018-08-30 12:19:36 +0200 |
commit | 8f6e1c37e3e74e1fd490a0de5360ff1467c521cd (patch) | |
tree | b6dd7190edd587d228a80781efccd302e958c8bb /src/parser_json.c | |
parent | 665da8df3cfaf482157b752f5de98c45038621c3 (diff) |
JSON: Rename mangle statement properties
Instead of the generic "left" and "right", use "key" and "value".
Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/parser_json.c')
-rw-r--r-- | src/parser_json.c | 46 |
1 files changed, 23 insertions, 23 deletions
diff --git a/src/parser_json.c b/src/parser_json.c index 1bb4f1f1..2f7e0a3e 100644 --- a/src/parser_json.c +++ b/src/parser_json.c @@ -1441,50 +1441,50 @@ static struct stmt *json_parse_verdict_stmt(struct json_ctx *ctx, } static struct stmt *json_parse_mangle_stmt(struct json_ctx *ctx, - const char *key, json_t *value) + const char *type, json_t *root) { - json_t *jleft, *jright; - struct expr *left, *right; + json_t *jkey, *jvalue; + struct expr *key, *value; struct stmt *stmt; - if (json_unpack_err(ctx, value, "{s:o, s:o}", - "left", &jleft, "right", &jright)) + if (json_unpack_err(ctx, root, "{s:o, s:o}", + "key", &jkey, "value", &jvalue)) return NULL; - left = json_parse_mangle_lhs_expr(ctx, jleft); - if (!left) { - json_error(ctx, "Invalid LHS of mangle statement"); + key = json_parse_mangle_lhs_expr(ctx, jkey); + if (!key) { + json_error(ctx, "Invalid mangle statement key"); return NULL; } - right = json_parse_stmt_expr(ctx, jright); - if (!right) { - json_error(ctx, "Invalid RHS of mangle statement"); - expr_free(left); + value = json_parse_stmt_expr(ctx, jvalue); + if (!value) { + json_error(ctx, "Invalid mangle statement value"); + expr_free(key); return NULL; } - switch (left->ops->type) { + switch (key->ops->type) { case EXPR_EXTHDR: - return exthdr_stmt_alloc(int_loc, left, right); + return exthdr_stmt_alloc(int_loc, key, value); case EXPR_PAYLOAD: - return payload_stmt_alloc(int_loc, left, right); + return payload_stmt_alloc(int_loc, key, value); case EXPR_META: - stmt = meta_stmt_alloc(int_loc, left->meta.key, right); - expr_free(left); + stmt = meta_stmt_alloc(int_loc, key->meta.key, value); + expr_free(key); return stmt; case EXPR_CT: - if (left->ct.key == NFT_CT_HELPER) { + if (key->ct.key == NFT_CT_HELPER) { stmt = objref_stmt_alloc(int_loc); stmt->objref.type = NFT_OBJECT_CT_HELPER; - stmt->objref.expr = right; + stmt->objref.expr = value; } else { - stmt = ct_stmt_alloc(int_loc, left->ct.key, - left->ct.direction, right); + stmt = ct_stmt_alloc(int_loc, key->ct.key, + key->ct.direction, value); } - expr_free(left); + expr_free(key); return stmt; default: - json_error(ctx, "Invalid LHS expression type for mangle statement."); + json_error(ctx, "Invalid mangle statement key expression type."); return NULL; } } |