No need to clone when assigning to the same variable. Fixes: 024500aeab9a ("expr: Evaluate the condition expression in a separate step.") Signed-off-by: Ilya Maximets <i.maxim...@ovn.org> --- tests/test-ovn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/test-ovn.c b/tests/test-ovn.c index 74b5a3384..18c675303 100644 --- a/tests/test-ovn.c +++ b/tests/test-ovn.c @@ -919,7 +919,7 @@ test_tree_shape_exhaustively(struct expr *expr, struct shash *symtab, } else if (operation >= OP_SIMPLIFY) { modified = expr_simplify(expr_clone(expr)); modified = expr_evaluate_condition( - expr_clone(modified), tree_shape_is_chassis_resident_cb, + modified, tree_shape_is_chassis_resident_cb, NULL, NULL); ovs_assert(expr_honors_invariants(modified)); -- 2.25.4 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev