learning-switch: Reserved addresses are destinations, not sources.

A switch is not supposed to forward packets directed to MAC addresses
01:80:c2:00:00:0x.  This code was instead dropping packets *from* those
addresses.

(This code is only used by ovs-controller, so the bug is not a big deal.)

diff --git a/lib/learning-switch.c b/lib/learning-switch.c
index 7eb9e1c29d0bb09e25de87d585368333122cced2..413a70fd6943b84e71afb11f98042fe38bd9637b 100644 (file)
--- a/lib/learning-switch.c
+++ b/lib/learning-switch.c
@@ -410,7 +410,8 @@ process_packet_in(struct lswitch *sw, struct rconn *rconn, void *opi_)
         }
     }
 
-    if (eth_addr_is_reserved(flow.dl_src)) {
+    /* Drop frames for reserved multicast addresses. */
+    if (eth_addr_is_reserved(flow.dl_dst)) {
         goto drop_it;
     }