Thanks, applied to master.
On Tue, Jan 08, 2013 at 01:32:38PM -0800, Ethan Jackson wrote: > Acked-by: Ethan Jackson <[email protected]> > > > On Wed, Dec 26, 2012 at 12:38 PM, Ben Pfaff <[email protected]> wrote: > > > This should be pretty minor because it only leaks memory if the match does > > not fit in the inline area of a miniflow and minimask. I imagine that the > > most common match used for an ofmonitor is "match everything", which > > wouldn't leak. > > > > Signed-off-by: Ben Pfaff <[email protected]> > > --- > > ofproto/connmgr.c | 1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/ofproto/connmgr.c b/ofproto/connmgr.c > > index 77b7b39..2e82f5d 100644 > > --- a/ofproto/connmgr.c > > +++ b/ofproto/connmgr.c > > @@ -1782,6 +1782,7 @@ void > > ofmonitor_destroy(struct ofmonitor *m) > > { > > if (m) { > > + minimatch_destroy(&m->match); > > hmap_remove(&m->ofconn->monitors, &m->ofconn_node); > > free(m); > > } > > -- > > 1.7.2.5 > > > > _______________________________________________ > > dev mailing list > > [email protected] > > http://openvswitch.org/mailman/listinfo/dev > > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
