brcompatd: Delete VLAN tags only for the correct port in del_port().
authorBen Pfaff <blp@nicira.com>
Mon, 21 Sep 2009 22:34:27 +0000 (15:34 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 22 Sep 2009 17:08:10 +0000 (10:08 -0700)
vlan.%s.* will match e.g. eth0.123 if the %s expands to eth0.  We only
want it to match eth0 in that case.

This is based on code inspection.  It may or may not fix a real problem.

vswitchd/ovs-brcompatd.c

index 50ed632a2fbdd126861c01a7daad1745e66c1516..13bb843dd624e5834fae8648414ab7a135f5520a 100644 (file)
@@ -525,7 +525,7 @@ del_port(const char *br_name, const char *port_name)
 {
     cfg_del_entry("bridge.%s.port=%s", br_name, port_name);
     cfg_del_match("bonding.*.slave=%s", port_name);
-    cfg_del_match("vlan.%s.*", port_name);
+    cfg_del_match("vlan.%s.[!0-9]*", port_name);
 }
 
 static int