Thanks. I tested this (in the process finding the use-after-free problem that I just sent out a fix for) and pushed the fix.
On Wed, Sep 28, 2011 at 11:39:57AM -0700, Ethan Jackson wrote: > Looks good. > > Ethan > > On Wed, Sep 28, 2011 at 09:37, Ben Pfaff <[email protected]> wrote: > > When an Interface record is invalid (for example, when the interface that > > it specifies does not exist and cannot be created), ovs-vswitchd would > > leave any pre-existing data in its columns, except that it would set the > > ofport column to -1 to indicate the error. ?This was sometimes confusing > > because, for example, the lacp_current field could still be set to "true" > > if LACP has previously been active and up-to-date. > > > > This commit changes ovs-vswitchd to reset all such data to its default > > values when an interface is invalid. > > > > Bug #7450. > > Reported-by: Duffie Cooley <[email protected]> > > Bug #7491. > > Reported-by: Ethan Jackson <[email protected]> > > Release Notes #7500. > > Reported-by: Keith Amidon <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
