[BRIDGE]: don't change packet type
authorStephen Hemminger <shemminger@linux-foundation.org>
Thu, 26 Apr 2007 05:03:10 +0000 (22:03 -0700)
committerDavid S. Miller <davem@sunset.davemloft.net>
Thu, 26 Apr 2007 05:30:00 +0000 (22:30 -0700)
The change to forward STP bpdu's (for usermode STP) through normal path,
changed the packet type in the process. Since link local stuff is multicast, it
should stay pkt_type = PACKET_MULTICAST.  The code was probably copy/pasted
incorrectly from the bridge pseudo-device receive path.

Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_input.c

index 8a55276..364e0ba 100644 (file)
@@ -131,12 +131,9 @@ struct sk_buff *br_handle_frame(struct net_bridge_port *p, struct sk_buff *skb)
        if (!is_valid_ether_addr(eth_hdr(skb)->h_source))
                goto drop;
 
-       if (unlikely(is_link_local(dest))) {
-               skb->pkt_type = PACKET_HOST;
-
+       if (unlikely(is_link_local(dest)))
                return (NF_HOOK(PF_BRIDGE, NF_BR_LOCAL_IN, skb, skb->dev,
                                NULL, br_handle_local_finish) == 0) ? skb : NULL;
-       }
 
        switch (p->state) {
        case BR_STATE_FORWARDING: