Blame SOURCES/0004-test-ovn-Fix-expression-leak.patch
|
 |
ebb439 |
From 06d9464335d81f4b78a4ff8c0daf890b947f571f Mon Sep 17 00:00:00 2001
|
|
|
ebb439 |
From: Ilya Maximets <i.maximets@ovn.org>
|
|
|
ebb439 |
Date: Fri, 20 Nov 2020 01:17:12 +0100
|
|
|
ebb439 |
Subject: [PATCH 04/16] test-ovn: Fix expression leak.
|
|
|
ebb439 |
|
|
|
ebb439 |
No need to clone when assigning to the same variable.
|
|
|
ebb439 |
|
|
|
ebb439 |
Fixes: 024500aeab9a ("expr: Evaluate the condition expression in a separate step.")
|
|
|
ebb439 |
Acked-by: Dumitru Ceara <dceara@redhat.com>
|
|
|
ebb439 |
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
|
|
|
ebb439 |
Signed-off-by: Numan Siddique <numans@ovn.org>
|
|
|
ebb439 |
---
|
|
|
ebb439 |
tests/test-ovn.c | 2 +-
|
|
|
ebb439 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
ebb439 |
|
|
|
ebb439 |
diff --git a/tests/test-ovn.c b/tests/test-ovn.c
|
|
|
ebb439 |
index 6662ced54..471126143 100644
|
|
|
ebb439 |
--- a/tests/test-ovn.c
|
|
|
ebb439 |
+++ b/tests/test-ovn.c
|
|
|
ebb439 |
@@ -918,7 +918,7 @@ test_tree_shape_exhaustively(struct expr *expr, struct shash *symtab,
|
|
|
ebb439 |
} else if (operation >= OP_SIMPLIFY) {
|
|
|
ebb439 |
modified = expr_simplify(expr_clone(expr));
|
|
|
ebb439 |
modified = expr_evaluate_condition(
|
|
|
ebb439 |
- expr_clone(modified), tree_shape_is_chassis_resident_cb,
|
|
|
ebb439 |
+ modified, tree_shape_is_chassis_resident_cb,
|
|
|
ebb439 |
NULL, NULL);
|
|
|
ebb439 |
ovs_assert(expr_honors_invariants(modified));
|
|
|
ebb439 |
|
|
|
ebb439 |
--
|
|
|
ebb439 |
2.28.0
|
|
|
ebb439 |
|