]> git.ipfire.org Git - thirdparty/nftables.git/commit
src: Fix dumping vlan rules
authorM. Braun <michael-dev@fami-braun.de>
Mon, 15 Jul 2019 16:59:01 +0000 (18:59 +0200)
committerPablo Neira Ayuso <pablo@netfilter.org>
Wed, 31 Jul 2019 12:44:46 +0000 (14:44 +0200)
commit88849ff15acaada4b0b76870ca48268236c6f30a
treeb771f7d7f3483f4d3d9a32dcb5b074b7124865b1
parenta79f5b729ede4fe39386f2c4971064083631fa7a
src: Fix dumping vlan rules

Given the following bridge rules:
1. ip protocol icmp accept
2. ether type vlan vlan type ip ip protocol icmp accept

The are currently both dumped by "nft list ruleset" as
1. ip protocol icmp accept
2. ip protocol icmp accept

Though, the netlink code actually is different

bridge filter FORWARD 4
  [ payload load 2b @ link header + 12 => reg 1 ]
  [ cmp eq reg 1 0x00000008 ]
  [ payload load 1b @ network header + 9 => reg 1 ]
  [ cmp eq reg 1 0x00000001 ]
  [ immediate reg 0 accept ]

bridge filter FORWARD 5 4
  [ payload load 2b @ link header + 12 => reg 1 ]
  [ cmp eq reg 1 0x00000081 ]
  [ payload load 2b @ link header + 16 => reg 1 ]
  [ cmp eq reg 1 0x00000008 ]
  [ payload load 1b @ network header + 9 => reg 1 ]
  [ cmp eq reg 1 0x00000001 ]
  [ immediate reg 0 accept ]

What happens here is that:

1. vlan type ip kills ether type vlan
2. ip protocol icmp kills vlan type ip

Fix this by avoiding the removal of all vlan statements
in the given example.

Signed-off-by: Michael Braun <michael-dev@fami-braun.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
src/payload.c
tests/py/bridge/vlan.t
tests/py/bridge/vlan.t.payload
tests/py/bridge/vlan.t.payload.netdev