diff mbox series

[net,v2,2/8] net: ipip: implement header_ops->parse_protocol for AF_PACKET

Message ID 20200630010625.469202-3-Jason@zx2c4.com
State New
Headers show
Series support AF_PACKET for layer 3 devices | expand

Commit Message

Jason A. Donenfeld June 30, 2020, 1:06 a.m. UTC
Ipip uses skb->protocol to determine packet type, and bails out if it's
not set. For AF_PACKET injection, we need to support its call chain of:

    packet_sendmsg -> packet_snd -> packet_parse_headers ->
      dev_parse_header_protocol -> parse_protocol

Without a valid parse_protocol, this returns zero, and ipip rejects the
skb. So, this wires up the ip_tunnel handler for layer 3 packets for
that case.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

---
 net/ipv4/ipip.c       | 1 +
 net/ipv6/ip6_tunnel.c | 1 +
 2 files changed, 2 insertions(+)

-- 
2.27.0

Comments

Willem de Bruijn June 30, 2020, 4:02 p.m. UTC | #1
On Mon, Jun 29, 2020 at 9:06 PM Jason A. Donenfeld <Jason@zx2c4.com> wrote:
>

> Ipip uses skb->protocol to determine packet type, and bails out if it's

> not set. For AF_PACKET injection, we need to support its call chain of:

>

>     packet_sendmsg -> packet_snd -> packet_parse_headers ->

>       dev_parse_header_protocol -> parse_protocol

>

> Without a valid parse_protocol, this returns zero, and ipip rejects the

> skb. So, this wires up the ip_tunnel handler for layer 3 packets for

> that case.

>

> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>


Acked-by: Willem de Bruijn <willemb@google.com>


For all protocols, really. But it probably doesn't add much to repeat
this in each patch.
diff mbox series

Patch

diff --git a/net/ipv4/ipip.c b/net/ipv4/ipip.c
index 40fea52c8277..75d35e76bec2 100644
--- a/net/ipv4/ipip.c
+++ b/net/ipv4/ipip.c
@@ -361,6 +361,7 @@  static const struct net_device_ops ipip_netdev_ops = {
 static void ipip_tunnel_setup(struct net_device *dev)
 {
 	dev->netdev_ops		= &ipip_netdev_ops;
+	dev->header_ops		= &ip_tunnel_header_ops;
 
 	dev->type		= ARPHRD_TUNNEL;
 	dev->flags		= IFF_NOARP;
diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
index 821d96c720b9..a18c378ca5f4 100644
--- a/net/ipv6/ip6_tunnel.c
+++ b/net/ipv6/ip6_tunnel.c
@@ -1846,6 +1846,7 @@  static const struct net_device_ops ip6_tnl_netdev_ops = {
 static void ip6_tnl_dev_setup(struct net_device *dev)
 {
 	dev->netdev_ops = &ip6_tnl_netdev_ops;
+	dev->header_ops = &ip_tunnel_header_ops;
 	dev->needs_free_netdev = true;
 	dev->priv_destructor = ip6_dev_free;