[PATCH] Fix SignalChange property.

David Ruth druth at chromium.org
Tue Jul 16 12:53:00 PDT 2024


SignalChange should be defined as a property of an interface.
Previously, it was incorrectly defined as a property of p2p peers.

Signed-off-by: David Ruth <druth at chromium.org>
---

 doc/dbus.doxygen               |  6 ++++++
 wpa_supplicant/dbus/dbus_new.c | 10 +++++-----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/doc/dbus.doxygen b/doc/dbus.doxygen
index 9f72614d4..070210945 100644
--- a/doc/dbus.doxygen
+++ b/doc/dbus.doxygen
@@ -1147,6 +1147,12 @@ fi.w1.wpa_supplicant1.CreateInterface.
       <li>
 	<h3>MACAddress - ay - (read)</h3>
 	<p>MAC address of the interface</p>
+      </li>
+
+      <li>
+	<h3>SignalChange - "a{sv}" - (read)</h3>
+	<p>Signal and quality properties of the interface</p>
+      </li>
     </ul>
 
 \subsection dbus_interface_signals Signals
diff --git a/wpa_supplicant/dbus/dbus_new.c b/wpa_supplicant/dbus/dbus_new.c
index ef7076934..76e42ffbc 100644
--- a/wpa_supplicant/dbus/dbus_new.c
+++ b/wpa_supplicant/dbus/dbus_new.c
@@ -4023,6 +4023,11 @@ static const struct wpa_dbus_property_desc wpas_dbus_interface_properties[] = {
 	  NULL,
 	  NULL,
 	},
+	{ "SignalChange", WPAS_DBUS_NEW_IFACE_INTERFACE, "a{sv}",
+	  wpas_dbus_getter_signal_change,
+	  NULL,
+	  NULL
+	},
 	{ NULL, NULL, NULL, NULL, NULL, NULL }
 };
 
@@ -4620,11 +4625,6 @@ static const struct wpa_dbus_property_desc wpas_dbus_p2p_peer_properties[] = {
 	  NULL,
 	  NULL
 	},
-	{ "SignalChange", WPAS_DBUS_NEW_IFACE_INTERFACE, "a{sv}",
-	  wpas_dbus_getter_signal_change,
-	  NULL,
-	  NULL
-	},
 	{ NULL, NULL, NULL, NULL, NULL, NULL }
 };
 
-- 
2.45.2.993.g49e7a77208-goog




More information about the Hostap mailing list