linux.git/net/bridge/br_input.c, branch v2.6.23

[BRIDGE]: Kill clone argument to br_flood_*

2007-09-16T23:20:48+00:00

The clone argument is only used by one caller and that caller can clone
the packet itself.  This patch moves the clone call into the caller and
kills the clone argument.

Signed-off-by: Herbert Xu 
Signed-off-by: David S. Miller

[BRIDGE]: Packets leaking out of disabled/blocked ports.

2007-08-31T05:15:35+00:00

This patch fixes some packet leakage in bridge.  The bridging code was
allowing forward table entries to be generated even if a device was
being blocked. The fix is to not add forwarding database entries
unless the port is active.

The bug arose as part of the conversion to processing STP frames
through normal receive path (in 2.6.17).

Signed-off-by: Stephen Hemminger 
Acked-by: John W. Linville 
Signed-off-by: David S. Miller

net/* misc endianness annotations

2007-07-26T18:11:56+00:00

Signed-off-by: Al Viro 
Signed-off-by: Linus Torvalds

[BRIDGE]: if no STP then forward all BPDUs

2007-04-26T05:30:02+00:00

If a bridge is not running STP, then it has no way to detect a cycle
in the network. But if it is not running STP and some other machine
or device is running STP, then if STP BPDU's get forwarded to it can
detect the cycle.

This is how the old 2.4 and early 2.6 code worked.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

[BRIDGE]: drop PAUSE frames

2007-04-26T05:30:01+00:00

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 
Signed-off-by: David S. Miller

[BRIDGE]: don't change packet type

2007-04-26T05:30:00+00:00

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 
Signed-off-by: David S. Miller

bridge: don't route packets while learning

2007-04-26T05:28:45+00:00

While in the STP learning state, don't route packets; wait until
forwarding delay has expired. The purpose of the forwarding delay
is to detect loops in the network, and if a brouter started up
and started forwarding, it could cause a flood.

Signed-off-by: Stephen Hemminger

bridge: eliminate call by reference

2007-04-26T05:28:44+00:00

Change the bridging hook to be simple function with return value
rather than modifying the skb argument. This could generate better
code and is cleaner.

Signed-off-by: Stephen Hemminger

[BRIDGE]: faster compare for link local addresses

2007-04-26T05:25:11+00:00

Use logic operations rather than memcmp() to compare destination
address with link local multicast addresses.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

[NET] BRIDGE: Fix whitespace errors.

2007-02-11T07:19:23+00:00

Signed-off-by: YOSHIFUJI Hideaki 
Signed-off-by: David S. Miller