netfilter-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH nft 0/3] use skb->protocol as l3 protocol dependency
@ 2019-06-18 18:43 Florian Westphal
  2019-06-18 18:43 ` [PATCH nft 1/3] netlink_delinerize: remove network header dep for reject statement also in bridge family Florian Westphal
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Florian Westphal @ 2019-06-18 18:43 UTC (permalink / raw)
  To: netfilter-devel

Real patch is the last one, first two do preparation work:
1. Allow removal of the dependency for the reject statement
2. Keep exact icmp type for bridge when printing.
   This is needed so we do not lose the l3 protocol information.
   In the ip family, "reject" and "reject with icmp type
   port-unreachable" are the same, but in case of bridge the latter
   adds a protocol dependency on ipv4, whereas the former rejects
   ip with icmp and ipv6 with a similar icmp-v6 error packet.
3. Prefer meta protocol for bridge family for all implicit
   depencencies.

 include/statement.h                   |    3 
 src/json.c                            |    6 -
 src/meta.c                            |    6 -
 src/netlink_delinearize.c             |   10 +
 src/payload.c                         |   18 +++
 src/statement.c                       |    6 -
 tests/py/bridge/ether.t               |    4 
 tests/py/bridge/ether.t.json.output   |   48 ---------
 tests/py/bridge/ether.t.payload       |   24 +++-
 tests/py/bridge/icmpX.t.payload       |    4 
 tests/py/bridge/reject.t              |   28 ++---
 tests/py/bridge/reject.t.json.output  |  170 +++++---------------------------
 tests/py/bridge/reject.t.payload      |   24 ++--
 tests/py/inet/ip_tcp.t.payload.bridge |    8 -
 tests/py/inet/sets.t.payload.bridge   |    4 
 tests/py/ip/ip.t.payload.bridge       |  180 +++++++++++++++++-----------------
 16 files changed, 217 insertions(+), 326 deletions(-)



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2019-06-19 17:35 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-18 18:43 [PATCH nft 0/3] use skb->protocol as l3 protocol dependency Florian Westphal
2019-06-18 18:43 ` [PATCH nft 1/3] netlink_delinerize: remove network header dep for reject statement also in bridge family Florian Westphal
2019-06-19 17:21   ` Pablo Neira Ayuso
2019-06-18 18:43 ` [PATCH nft 2/3] src: statement: disable reject statement type omission for bridge Florian Westphal
2019-06-19 17:21   ` Pablo Neira Ayuso
2019-06-18 18:43 ` [PATCH nft 3/3] src: prefer meta protocol as bridge l3 dependency Florian Westphal
2019-06-19 17:35   ` Pablo Neira Ayuso

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).