diff mbox series

EAP server: Add Ident field to MS-CHAP-Error

Message ID 20230209124450.2232-1-wu@domosekai.com
State Accepted
Headers show
Series EAP server: Add Ident field to MS-CHAP-Error | expand

Commit Message

Yihong Wu Feb. 9, 2023, 12:44 p.m. UTC
Per RFC2548, MS-CHAP-Error contains Ident field of one octet
followed by an ASCII message. The Ident field is missing.

Signed-off-by: Yihong Wu <wu@domosekai.com>
---
 src/eap_server/eap_server_ttls.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Jouni Malinen Feb. 20, 2023, 6:01 p.m. UTC | #1
On Thu, Feb 09, 2023 at 07:44:50AM -0500, Yihong Wu wrote:
> Per RFC2548, MS-CHAP-Error contains Ident field of one octet
> followed by an ASCII message. The Ident field is missing.

Thanks, applied.
diff mbox series

Patch

diff --git a/src/eap_server/eap_server_ttls.c b/src/eap_server/eap_server_ttls.c
index b89352244..b21f12da7 100644
--- a/src/eap_server/eap_server_ttls.c
+++ b/src/eap_server/eap_server_ttls.c
@@ -445,7 +445,8 @@  static struct wpabuf * eap_ttls_build_phase2_mschapv2(
 			sizeof(data->mschapv2_auth_response));
 	} else {
 		pos = eap_ttls_avp_hdr(pos, RADIUS_ATTR_MS_CHAP_ERROR,
-				       RADIUS_VENDOR_ID_MICROSOFT, 1, 6);
+				       RADIUS_VENDOR_ID_MICROSOFT, 1, 7);
+		*pos++ = data->mschapv2_ident;
 		os_memcpy(pos, "Failed", 6);
 		pos += 6;
 		AVP_PAD(req, pos);