From: Jouni Malinen <j@w1.fi>
Date: Sat, 13 Mar 2010 11:39:22 +0000 (+0200)
Subject: Fix WPS IE in Probe Response frame to include proper Config Methods values
X-Git-Tag: hostap_0_7_2~137
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=bdda27eb171893aaa8bf2f574fca517facda176b;p=thirdparty%2Fhostap.git

Fix WPS IE in Probe Response frame to include proper Config Methods values

This attribute is supposed to indicate which methods the AP supports as
an Enrollee for adding external Registrars. It was left to 0 when the
AP code did not yet support external Registrars and was forgotten when
the ER support was added.
---

diff --git a/src/wps/wps_registrar.c b/src/wps/wps_registrar.c
index 30283fe7f..2399d1c88 100644
--- a/src/wps/wps_registrar.c
+++ b/src/wps/wps_registrar.c
@@ -402,7 +402,11 @@ static int wps_build_probe_config_methods(struct wps_registrar *reg,
 					  struct wpabuf *msg)
 {
 	u16 methods;
-	methods = 0;
+	/*
+	 * These are the methods that the AP supports as an Enrollee for adding
+	 * external Registrars.
+	 */
+	methods = reg->wps->config_methods & ~WPS_CONFIG_PUSHBUTTON;
 	wpa_printf(MSG_DEBUG, "WPS:  * Config Methods (%x)", methods);
 	wpabuf_put_be16(msg, ATTR_CONFIG_METHODS);
 	wpabuf_put_be16(msg, 2);