tree df92ac74ffe075d30af2de1a280ee29b694f6dee
parent 178d940c8fa20666117208b3a6e40ed28579f122
author Hugo Benichi <hugobenichi@google.com> 1607923161 +0900
committer Commit Bot <commit-bot@chromium.org> 1608030364 +0000

patchpanel: correctly remove jump rules to traffic accounting chains

This patch fixes a mistake when adding or removing jump rules for
virtual or physical interfaces which was part of patch with Change-Id
I3f0e7feabea5a18f164b1fc78ef9628da4b353eb: the |op| argument was simply
ignored.

BUG=b:160112868
TEST=unit tests. Deployed patchpanel on rammus, connected and
disconnected a VPN multiple times and checked that jump rules don't
accumulate in iptables -v -t mangle -L POSTROUTING.

Change-Id: I9438eb0d97ccd3540ca2cc822352e24992e9cc97
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2586460
Tested-by: Hugo Benichi <hugobenichi@google.com>
Commit-Queue: Hugo Benichi <hugobenichi@google.com>
Reviewed-by: Jie Jiang <jiejiang@chromium.org>