dbus: Move roam metrics to the correct interface

These properties were in the wpas_dbus_bss_properties array when they
should have been in the wpas_dbus_interface_properties array. Move them
to the right place. This is the logical location for these properties
and it matches both the other parts of the implementation (e.g., being
in enum wpas_dbus_prop, not in enum wpas_dbus_bss_prop) and what
was originally documented for the interface in dbus.doxygen.

Fixes: 2bbad1c7c9cb ("dbus: Export roam time, roam complete, and session length")
Fixes: 80d06d0ca9f3 ("dbus: Export BSS Transition Management status")
Signed-off-by: Matthew Wang <matthewmwang@chromium.org>

diff --git a/wpa_supplicant/dbus/dbus_new.c b/wpa_supplicant/dbus/dbus_new.c
index 5e6b5220ece1cb2b0c75395345806972505bd1c0..e9e77bd182a05f44dcde71823648d0fbad38c12f 100644 (file)
--- a/wpa_supplicant/dbus/dbus_new.c
+++ b/wpa_supplicant/dbus/dbus_new.c
@@ -2855,30 +2855,6 @@ static const struct wpa_dbus_property_desc wpas_dbus_bss_properties[] = {
          NULL,
          NULL
        },
-       {
-         "RoamTime", WPAS_DBUS_NEW_IFACE_INTERFACE, "u",
-         wpas_dbus_getter_roam_time,
-         NULL,
-         NULL
-       },
-       {
-         "RoamComplete", WPAS_DBUS_NEW_IFACE_INTERFACE, "b",
-         wpas_dbus_getter_roam_complete,
-         NULL,
-         NULL
-       },
-       {
-         "SessionLength", WPAS_DBUS_NEW_IFACE_INTERFACE, "u",
-         wpas_dbus_getter_session_length,
-         NULL,
-         NULL
-       },
-       {
-         "BSSTMStatus", WPAS_DBUS_NEW_IFACE_INTERFACE, "u",
-         wpas_dbus_getter_bss_tm_status,
-         NULL,
-         NULL
-       },
        { NULL, NULL, NULL, NULL, NULL, NULL }
 };
 
@@ -3786,6 +3762,30 @@ static const struct wpa_dbus_property_desc wpas_dbus_interface_properties[] = {
          NULL,
          NULL
        },
+       {
+         "RoamTime", WPAS_DBUS_NEW_IFACE_INTERFACE, "u",
+         wpas_dbus_getter_roam_time,
+         NULL,
+         NULL
+       },
+       {
+         "RoamComplete", WPAS_DBUS_NEW_IFACE_INTERFACE, "b",
+         wpas_dbus_getter_roam_complete,
+         NULL,
+         NULL
+       },
+       {
+         "SessionLength", WPAS_DBUS_NEW_IFACE_INTERFACE, "u",
+         wpas_dbus_getter_session_length,
+         NULL,
+         NULL
+       },
+       {
+         "BSSTMStatus", WPAS_DBUS_NEW_IFACE_INTERFACE, "u",
+         wpas_dbus_getter_bss_tm_status,
+         NULL,
+         NULL
+       },
 #ifdef CONFIG_MESH
        { "MeshPeers", WPAS_DBUS_NEW_IFACE_MESH, "aay",
          wpas_dbus_getter_mesh_peers,