From 5035687a7be4c67e32b4e08319f2e66c76b42212 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Tue, 21 Jul 2020 13:51:17 +0200 Subject: [PATCH] core: only expose "type unicast" routes on D-Bus Currently, we would not mark non-unicast routes with their type, so they would wrongly appear as unicast routes in the D-Bus API. That is wrong. For now, just hide them. Fixes: 5d0d13f57010 ('platform: add support for local routes') --- src/nm-ip4-config.c | 3 +++ src/nm-ip6-config.c | 3 +++ 2 files changed, 6 insertions(+) diff --git a/src/nm-ip4-config.c b/src/nm-ip4-config.c index f017a3e809..9ad8116423 100644 --- a/src/nm-ip4-config.c +++ b/src/nm-ip4-config.c @@ -3227,6 +3227,9 @@ out_addresses_cached: nm_assert (_route_valid (route)); + if (route->type_coerced != nm_platform_route_type_coerce (RTN_UNICAST)) + continue; + g_variant_builder_init (&route_builder, G_VARIANT_TYPE ("a{sv}")); g_variant_builder_add (&route_builder, "{sv}", "dest", diff --git a/src/nm-ip6-config.c b/src/nm-ip6-config.c index 1589cad004..deb30e77d7 100644 --- a/src/nm-ip6-config.c +++ b/src/nm-ip6-config.c @@ -2682,6 +2682,9 @@ out_addresses_cached: nm_assert (_route_valid (route)); + if (route->type_coerced != nm_platform_route_type_coerce (RTN_UNICAST)) + continue; + g_variant_builder_init (&route_builder, G_VARIANT_TYPE ("a{sv}")); g_variant_builder_add (&route_builder, "{sv}", "dest",