[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