]> git.ipfire.org Git - thirdparty/strongswan.git/commit
eap-tls: Fix server implementation with TLS 1.2 and earlier
authorTobias Brunner <tobias@strongswan.org>
Mon, 13 Mar 2023 12:39:17 +0000 (13:39 +0100)
committerTobias Brunner <tobias@strongswan.org>
Tue, 21 Mar 2023 15:11:49 +0000 (16:11 +0100)
commit3d0d3f5d028a0c630f89cb4fec5b2cd5364f568b
tree4d9136b548e351ba8c3cad5bcf0f44bf37d862e1
parentdb87087fae6b4d1761dfba4eae3bd4880f5d23ab
eap-tls: Fix server implementation with TLS 1.2 and earlier

With 5401a74d3608 ("eap-tls: Add support for TLS 1.3") a TLS application
was added to implement TLS 1.3's protected success indication.  For
earlier TLS versions, its build() method simply returned SUCCESS as
there was nothing to send.  However, that had the unintended side-effect
of also not sending the final TLS handshake messages (ChangeCipherSpec
and Finished).

The reason is that the TLS stack first checks for remaining handshake
messages but then also asks the registered application for data to
piggyback to that response (before the commit there was no application,
so that step was skipped).  The problem is that the status returned by
the application is directly forwarded through the TLS stack.  So not
returning INVALID_STATE caused the session to get concluded immediately
instead of resulting in ALREADY_DONE that would trigger sending the
final EAP message instead of an EAP-Success.

Fixes: 5401a74d3608 ("eap-tls: Add support for TLS 1.3")
src/libcharon/plugins/eap_tls/eap_tls.c