Message ID | 1471880055-17832-1-git-send-email-zlpnobody@163.com |
---|---|
State | Accepted |
Delegated to: | Pablo Neira |
Headers | show |
On Mon, Aug 22, 2016 at 11:34:15PM +0800, Liping Zhang wrote: > From: Liping Zhang <liping.zhang@spreadtrum.com> > > We should keep existing syntax unchanged, and this was emphasized > in the commit 850f0a56b6ad ("src: add 'to' for snat and dnat")'s > commit log: "Existing syntax is still preserved, but the listing > shows the one including 'to'." > > This problem was found by running shell test: > # ./run-tests.sh > [ ... ] > W: [FAILED] ./testcases/maps/anonymous_snat_map_0 > I: [OK] ./testcases/maps/map_with_flags_0 > W: [FAILED] ./testcases/maps/named_snat_map_0 > [ ... ] Applied, thanks! -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/src/parser_bison.y b/src/parser_bison.y index 8025415..bf43608 100644 --- a/src/parser_bison.y +++ b/src/parser_bison.y @@ -1649,15 +1649,28 @@ stmt_expr : map_stmt_expr | primary_rhs_expr ; -nat_stmt_args : TO stmt_expr +nat_stmt_args : stmt_expr + { + $<stmt>0->nat.addr = $1; + } + | TO stmt_expr { $<stmt>0->nat.addr = $2; } + | stmt_expr COLON stmt_expr + { + $<stmt>0->nat.addr = $1; + $<stmt>0->nat.proto = $3; + } | TO stmt_expr COLON stmt_expr { $<stmt>0->nat.addr = $2; $<stmt>0->nat.proto = $4; } + | COLON stmt_expr + { + $<stmt>0->nat.proto = $2; + } | TO COLON stmt_expr { $<stmt>0->nat.proto = $3;