diff options
author | Stephen Hemminger <shemminger@linux-foundation.org> | 2007-04-26 01:05:55 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-04-26 01:30:01 -0400 |
commit | 2111f8b9e58fd04b87b8b07d66485f255a57b0bb (patch) | |
tree | aaeb558abcc903e8d4960368e7a4650c4ef6f4e9 /net/bridge | |
parent | 83aa0938ff59e8ef6d0b99260063ebe84fc84a16 (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/bridge')
-rw-r--r-- | net/bridge/br_input.c | 7 |
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: |