Fix a theoretical issue that would cause northd to include routes in the
flows of a Logical_Router that should not contain them.
This issue could only happen if there are two Logical_Router that share
the first part of their uuid.

Signed-off-by: Felix Huettner <[email protected]>
---
 northd/northd.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/northd/northd.c b/northd/northd.c
index 1097bb159..4f7f521f6 100644
--- a/northd/northd.c
+++ b/northd/northd.c
@@ -13767,6 +13767,9 @@ build_route_flows_for_lrouter(
     struct parsed_route *route;
     HMAP_FOR_EACH_WITH_HASH (route, key_node, uuid_hash(&od->key),
                              parsed_routes) {
+        if (od != route->od) {
+            continue;
+        }
         if (route->source == ROUTE_SOURCE_CONNECTED) {
             unique_routes_add(&unique_routes, route);
             continue;

base-commit: 5ebaf30b5a3c253c850e4ec4116b90f3c9077eb6
-- 
2.48.1

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to