From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Westphal Subject: [PATCH nft] parser: compact list of rhs keyword expressions Date: Wed, 27 Jul 2016 18:18:16 +0200 Message-ID: <1469636296-4452-1-git-send-email-fw@strlen.de> Cc: Florian Westphal To: Return-path: Received: from Chamillionaire.breakpoint.cc ([146.0.238.67]:34582 "EHLO Chamillionaire.breakpoint.cc" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757787AbcG0QSA (ORCPT ); Wed, 27 Jul 2016 12:18:00 -0400 Sender: netfilter-devel-owner@vger.kernel.org List-ID: Condenses the copy/pastry via a define, will make it less of a hassle to extend this list later if needed. Based on earlier patch from Pablo. Suggested-by: Pablo Neira Ayuso Signed-off-by: Florian Westphal --- src/parser_bison.y | 65 +++++++++++++----------------------------------------- 1 file changed, 15 insertions(+), 50 deletions(-) diff --git a/src/parser_bison.y b/src/parser_bison.y index 119fd09..31257fd 100644 --- a/src/parser_bison.y +++ b/src/parser_bison.y @@ -96,6 +96,8 @@ static void location_update(struct location *loc, struct location *rhs, int n) #define YYLLOC_DEFAULT(Current, Rhs, N) location_update(&Current, Rhs, N) +#define symbol_value(loc, str) \ + symbol_expr_alloc(loc, SYMBOL_VALUE, current_scope(state), str) %} /* Declaration section */ @@ -529,8 +531,8 @@ static void location_update(struct location *loc, struct location *rhs, int n) %type flow_key_expr flow_key_expr_alloc %destructor { expr_free($$); } flow_key_expr flow_key_expr_alloc -%type expr initializer_expr -%destructor { expr_free($$); } expr initializer_expr +%type expr initializer_expr keyword_expr +%destructor { expr_free($$); } expr initializer_expr keyword_expr %type rhs_expr concat_rhs_expr basic_rhs_expr %destructor { expr_free($$); } rhs_expr concat_rhs_expr basic_rhs_expr @@ -2166,38 +2168,19 @@ concat_rhs_expr : basic_rhs_expr } ; +keyword_expr : ETHER { $$ = symbol_value(&@$, "ether"); } + | IP { $$ = symbol_value(&@$, "ip"); } + | IP6 { $$ = symbol_value(&@$, "ip6"); } + | VLAN { $$ = symbol_value(&@$, "vlan"); } + | ARP { $$ = symbol_value(&@$, "arp"); } + | DNAT { $$ = symbol_value(&@$, "dnat"); } + | SNAT { $$ = symbol_value(&@$, "snat"); } + | ECN { $$ = symbol_value(&@$, "ecn"); } + ; + primary_rhs_expr : symbol_expr { $$ = $1; } | integer_expr { $$ = $1; } - | ETHER - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "ether"); - } - | IP - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "ip"); - } - | IP6 - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "ip6"); - } - | VLAN - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "vlan"); - } - | ARP - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "arp"); - } + | keyword_expr { $$ = $1; } | TCP { uint8_t data = IPPROTO_TCP; @@ -2275,24 +2258,6 @@ primary_rhs_expr : symbol_expr { $$ = $1; } BYTEORDER_HOST_ENDIAN, sizeof(data) * BITS_PER_BYTE, &data); } - | SNAT - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "snat"); - } - | DNAT - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "dnat"); - } - | ECN - { - $$ = symbol_expr_alloc(&@$, SYMBOL_VALUE, - current_scope(state), - "ecn"); - } ; relational_op : EQ { $$ = OP_EQ; } -- 2.7.3