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 <b...@ovn.org> --- lib/ovsdb-idl.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c index 7d3d328..7da25a5 100644 --- a/lib/ovsdb-idl.c +++ b/lib/ovsdb-idl.c @@ -748,7 +748,6 @@ ovsdb_idl_condition_reset(struct ovsdb_idl *idl, struct ovsdb_idl_table *table = ovsdb_idl_table_from_class(idl, tc); LIST_FOR_EACH_SAFE (c, next, node, &table->condition.clauses) { - ovs_list_remove(&c->node); ovsdb_idl_clause_free(c); } idl->cond_changed = table->cond_changed = true; -- 2.1.3 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev