summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorVishakha Narvekar <Vishakha.Narvekar@dell.com>2017-10-03 22:13:29 +0200
committerDavid S. Miller <davem@davemloft.net>2017-10-05 03:16:48 +0200
commite769fcec6bc4bdd1b0e2cf817680148f9c40b1c4 (patch)
tree11deb5a063028440de108c8779cf371469e23c37 /net
parentbpf: fix bpf_tail_call() x64 JIT (diff)
downloadlinux-e769fcec6bc4bdd1b0e2cf817680148f9c40b1c4.tar.xz
linux-e769fcec6bc4bdd1b0e2cf817680148f9c40b1c4.zip
net: 8021q: skip packets if the vlan is down
If the vlan is down, free the packet instead of proceeding with other processing, or counting it as received. If vlan interfaces are used as slaves for bonding, with arp monitoring for connectivity, if the rx counter is seen to be incrementing, then the bond device will not observe that the interface is down. CC: David S. Miller <davem@davemloft.net> Signed-off-by: Vishakha Narvekar <Vishakha.Narvekar@dell.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/8021q/vlan_core.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/net/8021q/vlan_core.c b/net/8021q/vlan_core.c
index e2ed69850489..0bc31de9071a 100644
--- a/net/8021q/vlan_core.c
+++ b/net/8021q/vlan_core.c
@@ -21,6 +21,12 @@ bool vlan_do_receive(struct sk_buff **skbp)
if (unlikely(!skb))
return false;
+ if (unlikely(!(vlan_dev->flags & IFF_UP))) {
+ kfree_skb(skb);
+ *skbp = NULL;
+ return false;
+ }
+
skb->dev = vlan_dev;
if (unlikely(skb->pkt_type == PACKET_OTHERHOST)) {
/* Our lower layer thinks this is not local, let's make sure.