]> git.ipfire.org Git - thirdparty/hostap.git/commit
EAP-TTLS peer: Fix parsing auth= and autheap= phase2 params
authorPali Rohár <pali.rohar@gmail.com>
Sun, 6 Dec 2015 11:01:32 +0000 (12:01 +0100)
committerJouni Malinen <j@w1.fi>
Thu, 17 Dec 2015 22:24:30 +0000 (00:24 +0200)
commitf24e48861d50b6b6fc5681f75d4aa75144862853
treeeabbec94eb90f5de2c27e85b14cd71430fd016d3
parent47c1de20a4352e7e3f7f0aeb9a936fd9a368275e
EAP-TTLS peer: Fix parsing auth= and autheap= phase2 params

This patch fixes an issue with an invalid phase2 parameter value
auth=MSCHAPv2 getting interpreted as auth=MSCHAP (v1) which could
degrade security (though, only within a protected TLS tunnel). Now when
invalid or unsupported auth= phase2 parameter combinations are
specified, EAP-TTLS initialization throws an error instead of silently
doing something.

More then one auth= phase2 type cannot be specified and also both auth= and
autheap= options cannot be specified.

Parsing phase2 type is case sensitive (as in other EAP parts), so phase2
parameter auth=MSCHAPv2 is invalid. Only auth=MSCHAPV2 is correct.

Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
[Use cstr_token() to get rid of unnecessary allocation; cleanup]
Signed-off-by: Jouni Malinen <j@w1.fi>
src/eap_peer/eap_ttls.c