|
 |
d37737 |
From 68392da523f43b9ae09f824fa68b04b20c9c88f5 Mon Sep 17 00:00:00 2001
|
|
|
d37737 |
From: Phil Sutter <psutter@redhat.com>
|
|
|
d37737 |
Date: Wed, 20 May 2020 11:12:37 +0200
|
|
|
d37737 |
Subject: [PATCH] parser_json: Support ranges in concat expressions
|
|
|
d37737 |
|
|
|
d37737 |
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1805798
|
|
|
d37737 |
Upstream Status: nftables commit 9475ca305a993
|
|
|
d37737 |
|
|
|
d37737 |
commit 9475ca305a993751b05cf26ef8e785a00de98b94
|
|
|
d37737 |
Author: Phil Sutter <phil@nwl.cc>
|
|
|
d37737 |
Date: Fri Mar 6 16:15:48 2020 +0100
|
|
|
d37737 |
|
|
|
d37737 |
parser_json: Support ranges in concat expressions
|
|
|
d37737 |
|
|
|
d37737 |
Duplicate commit 8ac2f3b2fca38's changes to bison parser into JSON
|
|
|
d37737 |
parser by introducing a new context flag signalling we're parsing
|
|
|
d37737 |
concatenated expressions.
|
|
|
d37737 |
|
|
|
d37737 |
Fixes: 8ac2f3b2fca38 ("src: Add support for concatenated set ranges")
|
|
|
d37737 |
Signed-off-by: Phil Sutter <phil@nwl.cc>
|
|
|
d37737 |
Acked-by: Eric Garver <eric@garver.life>
|
|
|
d37737 |
---
|
|
|
d37737 |
src/parser_json.c | 51 +++++++++++++++++++++++++++++----------------------
|
|
|
d37737 |
1 file changed, 29 insertions(+), 22 deletions(-)
|
|
|
d37737 |
|
|
|
d37737 |
diff --git a/src/parser_json.c b/src/parser_json.c
|
|
|
d37737 |
index 031930e..c48faa8 100644
|
|
|
d37737 |
--- a/src/parser_json.c
|
|
|
d37737 |
+++ b/src/parser_json.c
|
|
|
d37737 |
@@ -40,6 +40,7 @@
|
|
|
d37737 |
#define CTX_F_MANGLE (1 << 5)
|
|
|
d37737 |
#define CTX_F_SES (1 << 6) /* set_elem_expr_stmt */
|
|
|
d37737 |
#define CTX_F_MAP (1 << 7) /* LHS of map_expr */
|
|
|
d37737 |
+#define CTX_F_CONCAT (1 << 8) /* inside concat_expr */
|
|
|
d37737 |
|
|
|
d37737 |
struct json_ctx {
|
|
|
d37737 |
struct input_descriptor indesc;
|
|
|
d37737 |
@@ -99,6 +100,7 @@ static struct expr *json_parse_primary_expr(struct json_ctx *ctx, json_t *root);
|
|
|
d37737 |
static struct expr *json_parse_set_rhs_expr(struct json_ctx *ctx, json_t *root);
|
|
|
d37737 |
static struct expr *json_parse_set_elem_expr_stmt(struct json_ctx *ctx, json_t *root);
|
|
|
d37737 |
static struct expr *json_parse_map_lhs_expr(struct json_ctx *ctx, json_t *root);
|
|
|
d37737 |
+static struct expr *json_parse_concat_elem_expr(struct json_ctx *ctx, json_t *root);
|
|
|
d37737 |
static struct stmt *json_parse_stmt(struct json_ctx *ctx, json_t *root);
|
|
|
d37737 |
|
|
|
d37737 |
/* parsing helpers */
|
|
|
d37737 |
@@ -1058,7 +1060,7 @@ static struct expr *json_parse_concat_expr(struct json_ctx *ctx,
|
|
|
d37737 |
}
|
|
|
d37737 |
|
|
|
d37737 |
json_array_foreach(root, index, value) {
|
|
|
d37737 |
- tmp = json_parse_primary_expr(ctx, value);
|
|
|
d37737 |
+ tmp = json_parse_concat_elem_expr(ctx, value);
|
|
|
d37737 |
if (!tmp) {
|
|
|
d37737 |
json_error(ctx, "Parsing expr at index %zd failed.", index);
|
|
|
d37737 |
expr_free(expr);
|
|
|
d37737 |
@@ -1354,28 +1356,28 @@ static struct expr *json_parse_expr(struct json_ctx *ctx, json_t *root)
|
|
|
d37737 |
{ "set", json_parse_set_expr, CTX_F_RHS | CTX_F_STMT }, /* allow this as stmt expr because that allows set references */
|
|
|
d37737 |
{ "map", json_parse_map_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS },
|
|
|
d37737 |
/* below three are multiton_rhs_expr */
|
|
|
d37737 |
- { "prefix", json_parse_prefix_expr, CTX_F_RHS | CTX_F_STMT },
|
|
|
d37737 |
- { "range", json_parse_range_expr, CTX_F_RHS | CTX_F_STMT },
|
|
|
d37737 |
- { "payload", json_parse_payload_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "exthdr", json_parse_exthdr_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "tcp option", json_parse_tcp_option_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES },
|
|
|
d37737 |
- { "ip option", json_parse_ip_option_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES },
|
|
|
d37737 |
- { "meta", json_parse_meta_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "osf", json_parse_osf_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_MAP },
|
|
|
d37737 |
- { "ipsec", json_parse_xfrm_expr, CTX_F_PRIMARY | CTX_F_MAP },
|
|
|
d37737 |
- { "socket", json_parse_socket_expr, CTX_F_PRIMARY },
|
|
|
d37737 |
- { "rt", json_parse_rt_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "ct", json_parse_ct_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "numgen", json_parse_numgen_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
+ { "prefix", json_parse_prefix_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "range", json_parse_range_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "payload", json_parse_payload_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "exthdr", json_parse_exthdr_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "tcp option", json_parse_tcp_option_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "ip option", json_parse_ip_option_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "meta", json_parse_meta_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "osf", json_parse_osf_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "ipsec", json_parse_xfrm_expr, CTX_F_PRIMARY | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "socket", json_parse_socket_expr, CTX_F_PRIMARY | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "rt", json_parse_rt_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "ct", json_parse_ct_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_MANGLE | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "numgen", json_parse_numgen_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
/* below two are hash expr */
|
|
|
d37737 |
- { "jhash", json_parse_hash_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "symhash", json_parse_hash_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "fib", json_parse_fib_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "|", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "^", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "&", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { ">>", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
- { "<<", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP },
|
|
|
d37737 |
+ { "jhash", json_parse_hash_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "symhash", json_parse_hash_expr, CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "fib", json_parse_fib_expr, CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "|", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "^", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "&", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { ">>", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
+ { "<<", json_parse_binop_expr, CTX_F_RHS | CTX_F_STMT | CTX_F_PRIMARY | CTX_F_SET_RHS | CTX_F_SES | CTX_F_MAP | CTX_F_CONCAT },
|
|
|
d37737 |
{ "accept", json_parse_verdict_expr, CTX_F_RHS | CTX_F_SET_RHS },
|
|
|
d37737 |
{ "drop", json_parse_verdict_expr, CTX_F_RHS | CTX_F_SET_RHS },
|
|
|
d37737 |
{ "continue", json_parse_verdict_expr, CTX_F_RHS | CTX_F_SET_RHS },
|
|
|
d37737 |
@@ -1500,6 +1502,11 @@ static struct expr *json_parse_map_lhs_expr(struct json_ctx *ctx, json_t *root)
|
|
|
d37737 |
return json_parse_flagged_expr(ctx, CTX_F_MAP, root);
|
|
|
d37737 |
}
|
|
|
d37737 |
|
|
|
d37737 |
+static struct expr *json_parse_concat_elem_expr(struct json_ctx *ctx, json_t *root)
|
|
|
d37737 |
+{
|
|
|
d37737 |
+ return json_parse_flagged_expr(ctx, CTX_F_CONCAT, root);
|
|
|
d37737 |
+}
|
|
|
d37737 |
+
|
|
|
d37737 |
static struct expr *json_parse_dtype_expr(struct json_ctx *ctx, json_t *root)
|
|
|
d37737 |
{
|
|
|
d37737 |
if (json_is_string(root)) {
|
|
|
d37737 |
--
|
|
|
d37737 |
1.8.3.1
|
|
|
d37737 |
|