nat: Fix next feature for ED with multiple workers 95/28595/4
authorVladimir Isaev <visaev@netgate.com>
Thu, 27 Aug 2020 10:34:50 +0000 (13:34 +0300)
committerOle Tr�an <otroan@employees.org>
Fri, 11 Sep 2020 11:45:33 +0000 (11:45 +0000)
commit8fb4d10dc208fb3f284fe79e838343797cb2d813
tree4e4ba07e5dad1aa011b7655fc8c76d064439aee1
parent171d6aceb039a7f0b0d67c837ff74359dae01ae4
nat: Fix next feature for ED with multiple workers

Multiple (> 1) workers leads to handoff node being enabled.
This node pops next feature index to nat.arc_next to make sure
that packet will be pushed to the next feature in the arc.

But node nat44-ed-in2out-output also pops next feature and changes
arc_next. So actual next feature will be skipped in that case.

It leads to all nat44-ed-in2out packets being dropped if we have
multiple workers (handoff node enabled).

To resolve this a new node was added (nat-pre-in2out-output) to fill
arc_next in single worker case and multiple worker case is already
handled by handoff node.

Type: fix
Signed-off-by: Vladimir Isaev <visaev@netgate.com>
Change-Id: I9dfba68f00164d2d5ab867224871811bef4411ed
src/plugins/nat/in2out_ed.c
src/plugins/nat/nat.c
src/plugins/nat/nat.h