Nifty, Looks good other than that.
On Mon, Mar 21, 2011 at 1:25 PM, Ben Pfaff <[email protected]> wrote: > On Mon, Mar 21, 2011 at 01:22:58PM -0700, Ethan Jackson wrote: >> > ? ? } else { >> > ? ? ? ? struct bond_entry *e = lookup_bond_entry(port, flow, vlan); >> > - ? ? ? ?if (e->iface_idx < 0 || e->iface_idx >= port->n_ifaces >> > - ? ? ? ? ? ?|| !port->ifaces[e->iface_idx]->enabled) { >> > + ? ? ? ?if (!e->iface) { >> >> I think we still want to check if !e->iface->enabled here. > > Good catch, thank you! > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
