I was searching for a bug in the lacp stuff and thought it might be due to uninitialized memory so I ran through valgrind and exited with ovs-appctl exit. It was reported as a leak. There are a couple of other leaks actually, one or two which didn't matter, and one or two which I plan to investigate later. This one seemed the most serious however so I went ahead and fixed it.
Ethan On Tue, Mar 22, 2011 at 1:56 PM, Ben Pfaff <[email protected]> wrote: > On Tue, Mar 22, 2011 at 01:32:38PM -0700, Ethan Jackson wrote: >> --- a/vswitchd/bridge.c >> +++ b/vswitchd/bridge.c >> @@ -4207,6 +4207,7 @@ port_destroy(struct port *port) >> >> netdev_monitor_destroy(port->monitor); >> bitmap_free(port->trunks); >> + free(port->bond_hash); >> free(port->name); >> free(port); > > Looks good, thank you. > > (How did you notice this?) > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
