Thanks Ben. Ethan
On Sat, Aug 17, 2013 at 1:08 AM, Ben Pfaff <b...@nicira.com> wrote: > On Thu, Aug 15, 2013 at 06:37:41PM -0700, Ethan Jackson wrote: >> Reported-by: Ben Pfaff <b...@nicira.com> >> Signed-off-by: Ethan Jackson <et...@nicira.com> >> --- >> ofproto/ofproto-dpif-xlate.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c >> index 8be8088..a7f051d 100644 >> --- a/ofproto/ofproto-dpif-xlate.c >> +++ b/ofproto/ofproto-dpif-xlate.c >> @@ -289,6 +289,11 @@ xlate_remove_ofproto(struct ofproto_dpif *ofproto) >> } >> >> hmap_remove(&xbridges, &xbridge->hmap_node); >> + mac_learning_unref(xbridge->ml); >> + mbridge_unref(xbridge->mbridge); >> + dpif_sflow_unref(xbridge->sflow); >> + dpif_ipfix_unref(xbridge->ipfix); >> + stp_unref(xbridge->stp); > > I found that I also needed: > + hmap_destroy(&xbridge->xports); > here. > > I folded that in and applied this to master (since I think you're > probably sleeping). > >> free(xbridge->name); >> free(xbridge); >> } _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
