aboutsummaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorStephen Hemminger <shemminger@linux-foundation.org>2007-04-26 01:05:55 -0400
committerDavid S. Miller <davem@sunset.davemloft.net>2007-04-26 01:30:01 -0400
commit2111f8b9e58fd04b87b8b07d66485f255a57b0bb (patch)
treeaaeb558abcc903e8d4960368e7a4650c4ef6f4e9 /net
parent83aa0938ff59e8ef6d0b99260063ebe84fc84a16 (diff)
[BRIDGE]: drop PAUSE frames
Pause frames should never make it out of the network device into the stack. But if a device was misconfigured, it might happen. So drop pause frames in bridge. Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/bridge/br_input.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/net/bridge/br_input.c b/net/bridge/br_input.c
index 364e0ba44158..5662567c8aed 100644
--- a/net/bridge/br_input.c
+++ b/net/bridge/br_input.c
@@ -131,9 +131,14 @@ struct sk_buff *br_handle_frame(struct net_bridge_port *p, struct sk_buff *skb)
131 if (!is_valid_ether_addr(eth_hdr(skb)->h_source)) 131 if (!is_valid_ether_addr(eth_hdr(skb)->h_source))
132 goto drop; 132 goto drop;
133 133
134 if (unlikely(is_link_local(dest))) 134 if (unlikely(is_link_local(dest))) {
135 /* Pause frames shouldn't be passed up by driver anyway */
136 if (skb->protocol == htons(ETH_P_PAUSE))
137 goto drop;
138
135 return (NF_HOOK(PF_BRIDGE, NF_BR_LOCAL_IN, skb, skb->dev, 139 return (NF_HOOK(PF_BRIDGE, NF_BR_LOCAL_IN, skb, skb->dev,
136 NULL, br_handle_local_finish) == 0) ? skb : NULL; 140 NULL, br_handle_local_finish) == 0) ? skb : NULL;
141 }
137 142
138 switch (p->state) { 143 switch (p->state) {
139 case BR_STATE_FORWARDING: 144 case BR_STATE_FORWARDING: