Patchwork dbus: add missing signal description for WPS (7)

login
register
mail settings
Submitter Peter Wu
Date Feb. 4, 2013, 8:53 p.m.
Message ID <1561441.eQx7CLOFf7@al>
Download mbox | patch
Permalink /patch/218067/
State Accepted
Commit a24a5ccb28c2bc7af466e47b84dc346e6c54f062
Headers show

Comments

Peter Wu - Feb. 4, 2013, 8:53 p.m.
Commit caff399250311abfea3376b38530a88bb78b957d added a property named "WPS" to
the dbus interface. It did not add the new WPAS_DBUS_BSS_PROP_WPS property to
the function responsible for marking changed properties though. This cause the
following messages to be printed repeatedly to syslog:

    dbus: wpas_dbus_bss_signal_prop_changed: Unknown Property value 7

Signed-hostap: Peter Wu <lekensteyn@gmail.com>
---
Compile-tested, not with WPS hardware.
---
 wpa_supplicant/dbus/dbus_new.c | 3 +++
 1 file changed, 3 insertions(+)
Jouni Malinen - Feb. 9, 2013, 10:14 a.m.
On Mon, Feb 04, 2013 at 09:53:40PM +0100, Peter Wu wrote:
> Commit caff399250311abfea3376b38530a88bb78b957d added a property named "WPS" to
> the dbus interface. It did not add the new WPAS_DBUS_BSS_PROP_WPS property to
> the function responsible for marking changed properties though. This cause the
> following messages to be printed repeatedly to syslog:
> 
>     dbus: wpas_dbus_bss_signal_prop_changed: Unknown Property value 7

Thanks, applied.

Patch

diff --git a/wpa_supplicant/dbus/dbus_new.c b/wpa_supplicant/dbus/dbus_new.c
index 8bc6618..5b4a0a4 100644
--- a/wpa_supplicant/dbus/dbus_new.c
+++ b/wpa_supplicant/dbus/dbus_new.c
@@ -1808,6 +1808,9 @@  void wpas_dbus_bss_signal_prop_changed(struct wpa_supplicant *wpa_s,
 	case WPAS_DBUS_BSS_PROP_RSN:
 		prop = "RSN";
 		break;
+	case WPAS_DBUS_BSS_PROP_WPS:
+		prop = "WPS";
+		break;
 	case WPAS_DBUS_BSS_PROP_IES:
 		prop = "IEs";
 		break;