From patchwork Wed Feb 12 18:07:29 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kelsey Skunberg X-Patchwork-Id: 1236982 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=lists.ubuntu.com (client-ip=91.189.94.19; helo=huckleberry.canonical.com; envelope-from=kernel-team-bounces@lists.ubuntu.com; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=canonical.com Received: from huckleberry.canonical.com (huckleberry.canonical.com [91.189.94.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 48HngB4vsxz9sRR; Thu, 13 Feb 2020 05:07:50 +1100 (AEDT) Received: from localhost ([127.0.0.1] helo=huckleberry.canonical.com) by huckleberry.canonical.com with esmtp (Exim 4.86_2) (envelope-from ) id 1j1wQN-0003Ta-K0; Wed, 12 Feb 2020 18:07:47 +0000 Received: from youngberry.canonical.com ([91.189.89.112]) by huckleberry.canonical.com with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.86_2) (envelope-from ) id 1j1wQJ-0003SJ-8X for kernel-team@lists.ubuntu.com; Wed, 12 Feb 2020 18:07:43 +0000 Received: from mail-il1-f199.google.com ([209.85.166.199]) by youngberry.canonical.com with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.86_2) (envelope-from ) id 1j1wQI-0001lw-R7 for kernel-team@lists.ubuntu.com; Wed, 12 Feb 2020 18:07:42 +0000 Received: by mail-il1-f199.google.com with SMTP id c12so2316091ilr.18 for ; Wed, 12 Feb 2020 10:07:42 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=eHmBIvtY+Ozj7/6HU3Cc3JBEzCZE12ZE2Gk+ZD9K+KE=; b=t9qtCSiF3wJQcOxLOMqflmGu8spUxuSMvQ/kdDrXaR6VRujDEQkh51HawcmO8+zaIc y5PpEVk7mXOIiAu7WStf4bzLVq5aHYsytkInJ13L37iM0mGUt12l3uEDvRS7iXl6sUp7 RuCM/lFawFY4SvekBEjMAio9zue1kOObUuSjABiUmks/p/fpf1Otg1UtZyd5CgiPskB7 6Pf1wFmnRxOD6aqtW6q/BmLYBUKirW8sTyDovPyMc1VUh7UmdKPctv/DrYnEWSSUpack 5z1+I/mVgJZomsAeCtgdBxNxHQ/tst91X9GQKn5IbUe54FOYkPOnPn5WeSim5gyU+G52 Zycg== X-Gm-Message-State: APjAAAXMdUncK2ayv8xzqPp0hdsDEdSXetZ/wsDWfh0ackcqHZ822IQX udtOwNbSMUiWa6UcPj7ptQinklLJz0dFqiv1LrgiOGOw/+vOQjhwYIvMs4tyS1azrke5NBMqXdj wXzPxd0U6vMLlQZ6PMlb2XJdHnVo3DsNGafvk504bCw== X-Received: by 2002:a92:84ce:: with SMTP id y75mr11909541ilk.93.1581530861734; Wed, 12 Feb 2020 10:07:41 -0800 (PST) X-Google-Smtp-Source: APXvYqxdsPiYyeNituFlXs5OLz8W+T3UsU0lwzlujkreUl7XASwuDt9mWI2YrcIVPfHUnxMg29cFLA== X-Received: by 2002:a92:84ce:: with SMTP id y75mr11909523ilk.93.1581530861534; Wed, 12 Feb 2020 10:07:41 -0800 (PST) Received: from localhost.localdomain (c-73-243-191-173.hsd1.co.comcast.net. [73.243.191.173]) by smtp.gmail.com with ESMTPSA id a21sm259044ioh.29.2020.02.12.10.07.41 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 Feb 2020 10:07:41 -0800 (PST) From: Kelsey Skunberg To: kernel-team@lists.ubuntu.com Subject: [SRU][F/E/B/X][PATCH v2 2/2] xfrm interface: fix packet tx through bpf_redirect() Date: Wed, 12 Feb 2020 11:07:29 -0700 Message-Id: <20200212180729.10428-3-kelsey.skunberg@canonical.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20200212180729.10428-1-kelsey.skunberg@canonical.com> References: <20200212180729.10428-1-kelsey.skunberg@canonical.com> MIME-Version: 1.0 X-BeenThere: kernel-team@lists.ubuntu.com X-Mailman-Version: 2.1.20 Precedence: list List-Id: Kernel team discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kernel-team-bounces@lists.ubuntu.com Sender: "kernel-team" From: Nicolas Dichtel BugLink: https://bugs.launchpad.net/bugs/1860969 With an ebpf program that redirects packets through a xfrm interface, packets are dropped because no dst is attached to skb. This could also be reproduced with an AF_PACKET socket, with the following python script (xfrm1 is a xfrm interface): import socket send_s = socket.socket(socket.AF_PACKET, socket.SOCK_RAW, 0) # scapy # p = IP(src='10.100.0.2', dst='10.200.0.1')/ICMP(type='echo-request') # raw(p) req = b'E\x00\x00\x1c\x00\x01\x00\x00@\x01e\xb2\nd\x00\x02\n\xc8\x00\x01\x08\x00\xf7\xff\x00\x00\x00\x00' send_s.sendto(req, ('xfrm1', 0x800, 0, 0)) It was also not possible to send an ip packet through an AF_PACKET socket because a LL header was expected. Let's remove those LL header constraints. Signed-off-by: Nicolas Dichtel Signed-off-by: Steffen Klassert (cherry picked from commit f042365dbffea98fb8148c98c700402e8d099f02) Signed-off-by: Kelsey Skunberg --- net/xfrm/xfrm_interface.c | 32 +++++++++++++++++++++++++------- 1 file changed, 25 insertions(+), 7 deletions(-) diff --git a/net/xfrm/xfrm_interface.c b/net/xfrm/xfrm_interface.c index 2ab4859df55a..6809b96714e5 100644 --- a/net/xfrm/xfrm_interface.c +++ b/net/xfrm/xfrm_interface.c @@ -268,9 +268,6 @@ xfrmi_xmit2(struct sk_buff *skb, struct net_device *dev, struct flowi *fl) int err = -1; int mtu; - if (!dst) - goto tx_err_link_failure; - dst_hold(dst); dst = xfrm_lookup_with_ifid(xi->net, dst, fl, NULL, 0, xi->p.if_id); if (IS_ERR(dst)) { @@ -343,6 +340,7 @@ static netdev_tx_t xfrmi_xmit(struct sk_buff *skb, struct net_device *dev) { struct xfrm_if *xi = netdev_priv(dev); struct net_device_stats *stats = &xi->dev->stats; + struct dst_entry *dst = skb_dst(skb); struct flowi fl; int ret; @@ -352,10 +350,33 @@ static netdev_tx_t xfrmi_xmit(struct sk_buff *skb, struct net_device *dev) case htons(ETH_P_IPV6): xfrm_decode_session(skb, &fl, AF_INET6); memset(IP6CB(skb), 0, sizeof(*IP6CB(skb))); + if (!dst) { + fl.u.ip6.flowi6_oif = dev->ifindex; + fl.u.ip6.flowi6_flags |= FLOWI_FLAG_ANYSRC; + dst = ip6_route_output(dev_net(dev), NULL, &fl.u.ip6); + if (dst->error) { + dst_release(dst); + stats->tx_carrier_errors++; + goto tx_err; + } + skb_dst_set(skb, dst); + } break; case htons(ETH_P_IP): xfrm_decode_session(skb, &fl, AF_INET); memset(IPCB(skb), 0, sizeof(*IPCB(skb))); + if (!dst) { + struct rtable *rt; + + fl.u.ip4.flowi4_oif = dev->ifindex; + fl.u.ip4.flowi4_flags |= FLOWI_FLAG_ANYSRC; + rt = __ip_route_output_key(dev_net(dev), &fl.u.ip4); + if (IS_ERR(rt)) { + stats->tx_carrier_errors++; + goto tx_err; + } + skb_dst_set(skb, &rt->dst); + } break; default: goto tx_err; @@ -563,12 +584,9 @@ static void xfrmi_dev_setup(struct net_device *dev) { dev->netdev_ops = &xfrmi_netdev_ops; dev->type = ARPHRD_NONE; - dev->hard_header_len = ETH_HLEN; - dev->min_header_len = ETH_HLEN; dev->mtu = ETH_DATA_LEN; dev->min_mtu = ETH_MIN_MTU; - dev->max_mtu = ETH_DATA_LEN; - dev->addr_len = ETH_ALEN; + dev->max_mtu = IP_MAX_MTU; dev->flags = IFF_NOARP; dev->needs_free_netdev = true; dev->priv_destructor = xfrmi_dev_free;