Thanks. I pushed this to master. The leak is present on other branches too but it's so minor that I'm not sure it's worth backporting.
On Wed, Aug 15, 2012 at 05:35:47PM -0700, Mehak Mahajan wrote: > Hey Ben, > > Looks good. > > thanx! > mehak > > On Wed, Aug 15, 2012 at 4:19 PM, Ben Pfaff <b...@nicira.com> wrote: > > > Found by valgrind. > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > --- > > lib/ovsdb-idl.c | 1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c > > index 534beb0..6118852 100644 > > --- a/lib/ovsdb-idl.c > > +++ b/lib/ovsdb-idl.c > > @@ -230,6 +230,7 @@ ovsdb_idl_destroy(struct ovsdb_idl *idl) > > json_destroy(idl->monitor_request_id); > > free(idl->lock_name); > > json_destroy(idl->lock_request_id); > > + hmap_destroy(&idl->outstanding_txns); > > free(idl); > > } > > } > > -- > > 1.7.2.5 > > > > _______________________________________________ > > dev mailing list > > dev@openvswitch.org > > http://openvswitch.org/mailman/listinfo/dev > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
