I think so, too.
However, valgrind does not report any leak related to 'classes'. So I don't
have much confidence.

On Fri, Jan 22, 2016 at 8:39 AM, Ben Pfaff <b...@ovn.org> wrote:

> On Fri, Jan 22, 2016 at 08:29:27AM -0800, William Tu wrote:
> > Testcase 1311, 1312: Boolean-distinct queries on scalars, reports leak
> > below:
> >     xmalloc (util.c:112)
> >     do_query_distinct (test-ovsdb.c:1208)
> >     ovs_cmdl_run_command (command-line.c:121)
> >     main (test-ovsdb.c:72)
> >
> > Signed-off-by: William Tu <u9012...@gmail.com>
> > ---
> >  tests/test-ovsdb.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/test-ovsdb.c b/tests/test-ovsdb.c
> > index 15f41b0..7b774c2 100644
> > --- a/tests/test-ovsdb.c
> > +++ b/tests/test-ovsdb.c
> > @@ -1298,6 +1298,7 @@ do_query_distinct(struct ovs_cmdl_context *ctx)
> >
> >      ovsdb_table_destroy(table); /* Also destroys 'ts'. */
> >
> > +    free(rows);
> >      exit(exit_code);
> >  }
>
> Isn't the 'classes' variable in the same function also leaked?
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to