redirects: properly handle src_dport in SNAT rules
authorHans Dedecker <dedeckeh@gmail.com>
Thu, 6 Dec 2018 17:03:06 +0000 (18:03 +0100)
committerHans Dedecker <dedeckeh@gmail.com>
Thu, 6 Dec 2018 20:16:03 +0000 (21:16 +0100)
commit14589c80cde937162da02414a0103653a566e866
tree803ead07d0804932a2175232de552edc544d8bd8
parent1c4d5bcd1137e61e91dca858fe33d76d7a1dc821
redirects: properly handle src_dport in SNAT rules

In case of SNAT rules the src_dport parameter is used both as a rewrite
parameter as well as a matching parameter which is not the expected
behavior.
The latter is caused by port_redir being set to src_dport in case dest_port
parameter is not.
As this logic is in place to mimic the old shell script based firewall
behavior for DNAT only set port_redir in case the redirect rule is
a DNAT rule.

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>
Acked-by: Jo-Philipp Wich <jo@mein.io>
redirects.c