]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
vxlan: don't collect metadata if remote checksum is wrong
authorFabian Frederick <fabf@skynet.be>
Fri, 25 Sep 2020 13:16:02 +0000 (15:16 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 25 Sep 2020 23:58:07 +0000 (16:58 -0700)
call vxlan_remcsum() before md filling in vxlan_rcv()

Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/vxlan.c

index b9fefe27e3e898452ecb05b486bc1dcd59a1622b..47c762f7f5b11d8360c732339d75ea5bb6089720 100644 (file)
@@ -1875,6 +1875,10 @@ static int vxlan_rcv(struct sock *sk, struct sk_buff *skb)
                                   !net_eq(vxlan->net, dev_net(vxlan->dev))))
                goto drop;
 
+       if (vs->flags & VXLAN_F_REMCSUM_RX)
+               if (!vxlan_remcsum(&unparsed, skb, vs->flags))
+                       goto drop;
+
        if (vxlan_collect_metadata(vs)) {
                struct metadata_dst *tun_dst;
 
@@ -1891,9 +1895,6 @@ static int vxlan_rcv(struct sock *sk, struct sk_buff *skb)
                memset(md, 0, sizeof(*md));
        }
 
-       if (vs->flags & VXLAN_F_REMCSUM_RX)
-               if (!vxlan_remcsum(&unparsed, skb, vs->flags))
-                       goto drop;
        if (vs->flags & VXLAN_F_GBP)
                vxlan_parse_gbp_hdr(&unparsed, skb, vs->flags, md);
        /* Note that GBP and GPE can never be active together. This is