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
