On 2/12/25 5:12 PM, Felix Huettner via dev wrote: > 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; >
Thanks, Felix! Applied to main. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
