Re: [ovs-dev] [PATCH] ovsdb-idl: Fix double-remove in ovsdb_idl_condition_reset().

2016-08-15 Thread Ben Pfaff
On Mon, Aug 15, 2016 at 01:40:36PM -0700, Andy Zhou wrote: > On Sat, Aug 13, 2016 at 5:37 PM, Ben Pfaff wrote: > > > Both ovsdb_idl_condition_reset() and ovsdb_idl_clause_free() call > > ovs_list_remove() on the clause's 'node' member, but it should only be > > called once. > > > > Signed-off-by:

Re: [ovs-dev] [PATCH] ovsdb-idl: Fix double-remove in ovsdb_idl_condition_reset().

2016-08-15 Thread Andy Zhou
On Sat, Aug 13, 2016 at 5:37 PM, Ben Pfaff wrote: > Both ovsdb_idl_condition_reset() and ovsdb_idl_clause_free() call > ovs_list_remove() on the clause's 'node' member, but it should only be > called once. > > Signed-off-by: Ben Pfaff > Acked-by: Andy Zhou

[ovs-dev] [PATCH] ovsdb-idl: Fix double-remove in ovsdb_idl_condition_reset().

2016-08-13 Thread Ben Pfaff
Both ovsdb_idl_condition_reset() and ovsdb_idl_clause_free() call ovs_list_remove() on the clause's 'node' member, but it should only be called once. Signed-off-by: Ben Pfaff --- lib/ovsdb-idl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c index 7d3d328..7