> > + if (unlikely(skb->protocol = htons(ETH_P_PAE))) > > + goto drop; > > + > > switch (p->state) { > > case BR_STATE_FORWARDING: > > Not needed because the bridge is already handling it: > > 1) If running STP (ie true bridge), then all link local multicast is only received by > the bridge and never forwarded. Well, typical access point setups bridge the wireless AP interface with wired, EAPOL frames can be unicast (and 802.11 specifies to do so) and we want to avoid having them unicast to another host. Also, 802.1X in C.3.3 recommends not bridging the *ethertype* rather than depending on the link-local multicast address because otherwise eapol frames can be unicast into the network behind the (authorized) port which is undesirable. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Bridge mailing list Bridge@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/bridge