Fixed interoperability issue with PEAPv0 cryptobinding and NPS
authorJouni Malinen <j@w1.fi>
Sun, 14 Dec 2008 22:15:54 +0000 (00:15 +0200)
committerJouni Malinen <j@w1.fi>
Sun, 14 Dec 2008 22:15:54 +0000 (00:15 +0200)
commitf4f2774a96d91753811e921380856bc859888153
treef4513be9150e8ee13b2777a7690f2073f91e65cb
parent000a1de72b20a461710667477b98618ad545e941
Fixed interoperability issue with PEAPv0 cryptobinding and NPS

Windows Server 2008 NPS gets very confused if the TLS Message Length is
not included in the Phase 1 messages even if fragmentation is not used.
If the TLS Message Length field is not included in ClientHello message,
NPS seems to decide to use the ClientHello data (excluding first six
octets, i.e., EAP header, type, Flags) as the OuterTLVs data in
Cryptobinding Compound_MAC calculation (per PEAPv2; not MS-PEAP)..

Lets add the TLS Message Length to PEAPv0 Phase 1 messages to get rid of
this issue. This seems to fix Cryptobinding issues with NPS and PEAPv0
is now using optional Cryptobinding by default (again) since there are
no known interop issues with it anymore.
src/eap_peer/eap_peap.c
src/eap_peer/eap_tls_common.c
wpa_supplicant/ChangeLog