fixes for 4.14

Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/queue-4.14/macvlan-use-skb_reset_mac_header-in-macvlan_queue_xm.patch b/queue-4.14/macvlan-use-skb_reset_mac_header-in-macvlan_queue_xm.patch
new file mode 100644 (file)
index 0000000..0622cbf
--- /dev/null
+++ b/queue-4.14/macvlan-use-skb_reset_mac_header-in-macvlan_queue_xm.patch
@@ -0,0 +1,54 @@
+From a4d6634a34bd1e65256debf50babf1cce14ff739 Mon Sep 17 00:00:00 2001
+From: Sasha Levin <sashal@kernel.org>
+Date: Tue, 14 Jan 2020 13:00:35 -0800
+Subject: macvlan: use skb_reset_mac_header() in macvlan_queue_xmit()
+
+From: Eric Dumazet <edumazet@google.com>
+
+[ Upstream commit 1712b2fff8c682d145c7889d2290696647d82dab ]
+
+I missed the fact that macvlan_broadcast() can be used both
+in RX and TX.
+
+skb_eth_hdr() makes only sense in TX paths, so we can not
+use it blindly in macvlan_broadcast()
+
+Fixes: 96cc4b69581d ("macvlan: do not assume mac_header is set in macvlan_broadcast()")
+Signed-off-by: Eric Dumazet <edumazet@google.com>
+Reported-by: Jurgen Van Ham <juvanham@gmail.com>
+Tested-by: Matteo Croce <mcroce@redhat.com>
+Signed-off-by: David S. Miller <davem@davemloft.net>
+Signed-off-by: Sasha Levin <sashal@kernel.org>
+---
+ drivers/net/macvlan.c | 5 +++--
+ 1 file changed, 3 insertions(+), 2 deletions(-)
+
+diff --git a/drivers/net/macvlan.c b/drivers/net/macvlan.c
+index 2b977655834c..ab539136d5bf 100644
+--- a/drivers/net/macvlan.c
++++ b/drivers/net/macvlan.c
+@@ -263,7 +263,7 @@ static void macvlan_broadcast(struct sk_buff *skb,
+                             struct net_device *src,
+                             enum macvlan_mode mode)
+ {
+-      const struct ethhdr *eth = skb_eth_hdr(skb);
++      const struct ethhdr *eth = eth_hdr(skb);
+       const struct macvlan_dev *vlan;
+       struct sk_buff *nskb;
+       unsigned int i;
+@@ -515,10 +515,11 @@ static int macvlan_queue_xmit(struct sk_buff *skb, struct net_device *dev)
+       const struct macvlan_dev *dest;
+ 
+       if (vlan->mode == MACVLAN_MODE_BRIDGE) {
+-              const struct ethhdr *eth = (void *)skb->data;
++              const struct ethhdr *eth = skb_eth_hdr(skb);
+ 
+               /* send to other bridge ports directly */
+               if (is_multicast_ether_addr(eth->h_dest)) {
++                      skb_reset_mac_header(skb);
+                       macvlan_broadcast(skb, port, dev, MACVLAN_MODE_BRIDGE);
+                       goto xmit_world;
+               }
+-- 
+2.20.1
+

diff --git a/queue-4.14/series b/queue-4.14/series
index 6aabbdd8faaa5d6d582bf2b7adfeed1be95fef38..031d8bc6817eea3846a35a8a2adde4ce027d289c 100644 (file)
--- a/queue-4.14/series
+++ b/queue-4.14/series
@@ -38,3 +38,4 @@ netfilter-fix-a-use-after-free-in-mtype_destroy.patch
 netfilter-arp_tables-init-netns-pointer-in-xt_tgdtor_param-struct.patch
 nfc-pn533-fix-bulk-message-timeout.patch
 batman-adv-fix-dat-candidate-selection-on-little-endian-systems.patch
+macvlan-use-skb_reset_mac_header-in-macvlan_queue_xm.patch