[ovs-dev] [PATCH v2] extend-table: Fix a bug that iterates wrong table

Yifeng Sun pkusunyifeng at gmail.com
Tue Oct 2 18:37:03 UTC 2018


This seems to be a copy and paste bug that iterates and frees
the wrong table. This commit fixes that.

Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10730
Signed-off-by: Yifeng Sun <pkusunyifeng at gmail.com>
---
v1->v2: Fix email subject by adding [ovs-dev]

 ovn/lib/extend-table.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ovn/lib/extend-table.c b/ovn/lib/extend-table.c
index 511d1a84b0c0..aba0b2e68b2c 100644
--- a/ovn/lib/extend-table.c
+++ b/ovn/lib/extend-table.c
@@ -39,8 +39,8 @@ ovn_extend_table_destroy(struct ovn_extend_table *table)
     bitmap_free(table->table_ids);
 
     struct ovn_extend_table_info *desired, *d_next;
-    HMAP_FOR_EACH_SAFE (desired, d_next, hmap_node, &table->existing) {
-        hmap_remove(&table->existing, &desired->hmap_node);
+    HMAP_FOR_EACH_SAFE (desired, d_next, hmap_node, &table->desired) {
+        hmap_remove(&table->desired, &desired->hmap_node);
         free(desired->name);
         free(desired);
     }
-- 
2.7.4



More information about the dev mailing list