Hi Ben, This seems to take care of datums of type is_map, do we need to need to the same for type is_set()?
Thanks Amitabha > On Aug 26, 2016, at 9:03 AM, Ben Pfaff <b...@ovn.org> wrote: > > Found by inspection. > > Signed-off-by: Ben Pfaff <b...@ovn.org> > --- > ovsdb/ovsdb-idlc.in | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/ovsdb/ovsdb-idlc.in b/ovsdb/ovsdb-idlc.in > index 487876b..40be763 100755 > --- a/ovsdb/ovsdb-idlc.in > +++ b/ovsdb/ovsdb-idlc.in > @@ -957,6 +957,8 @@ void > function, > &%(s)s_columns[%(S)s_COL_%(C)s], > &datum); > + > + ovsdb_datum_destroy(&datum, &%(s)s_col_%(c)s.type); > } > """ % {'t': tableName, > 'T': tableName.upper(), > @@ -1123,6 +1125,8 @@ void > function, > &%(s)s_columns[%(S)s_COL_%(C)s], > &datum); > + > + ovsdb_datum_destroy(&datum, &%(s)s_col_%(c)s.type); > } > """ % {'t': tableName, > 'T': tableName.upper(), > -- > 2.1.3 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev