vswitch: Keep list of old interfaces sorted when reconfiguring ports.
authorBen Pfaff <blp@nicira.com>
Wed, 15 Apr 2009 22:03:14 +0000 (15:03 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 15 Apr 2009 22:03:14 +0000 (15:03 -0700)
The old_ifaces array must be sorted because we apply svec_contains() to
it, but there was nothing guaranteeing that it was sorted.  So add a
call to svec_sort().

vswitchd/bridge.c

index eed105c20ea1965c5dc06b76d3c15727d1d4b915..dc51e20eb5670a3bf5a3b8a717491894cafc1031 100644 (file)
@@ -1844,6 +1844,7 @@ port_reconfigure(struct port *port)
     for (i = 0; i < port->n_ifaces; i++) {
         svec_add(&old_ifaces, port->ifaces[i]->name);
     }
+    svec_sort(&old_ifaces);
     if (bonded) {
         cfg_get_all_keys(&new_ifaces, "bonding.%s.slave", port->name);
         if (!new_ifaces.n) {