DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

OpenSSL: Terminate TLS handshake if ClientHello cannot be generated

Browse source 
OpenSSL 1.1.1 added cases where ClientHello generation may fail due to
"no ciphers available". There is no point in sending out the resulting
TLS Alert message to the server since the server does not know what to
do with it before ClientHello. Instead, simply terminate the TLS
handshake locally and report EAP failure to avoid getting stuck waiting
for a timeout.

Signed-off-by: Jouni Malinen <j@w1.fi>
This commit is contained in:
Jouni Malinen 2018-05-01 12:41:28 +03:00
parent a61ee84d0c
commit 4a576c4736
1 changed files with 15 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/crypto/tls_openssl.c
View file

@ -232,6 +232,7 @@ struct tls_connection {
unsigned int server_cert_only:1; unsigned int server_cert_only:1;
unsigned int invalid_hb_used:1; unsigned int invalid_hb_used:1;
unsigned int success_data:1; unsigned int success_data:1;
unsigned int client_hello_generated:1;
u8 srv_cert_hash[32]; u8 srv_cert_hash[32];
@ -3597,8 +3598,22 @@ openssl_handshake(struct tls_connection *conn, const struct wpabuf *in_data,
else { else {
tls_show_errors(MSG_INFO, __func__, "SSL_connect"); tls_show_errors(MSG_INFO, __func__, "SSL_connect");
conn->failed++; conn->failed++;
if (!server && !conn->client_hello_generated) {
/* The server would not understand TLS Alert
* before ClientHello, so simply terminate
* handshake on this type of error case caused
* by a likely internal error like no ciphers
* available. */
wpa_printf(MSG_DEBUG,
"OpenSSL: Could not generate ClientHello");
conn->write_alerts++;
return NULL;
} }
} }
}
if (!server && !conn->failed)
conn->client_hello_generated = 1;
#ifdef CONFIG_SUITEB #ifdef CONFIG_SUITEB
if ((conn->flags & TLS_CONN_SUITEB) && !server && if ((conn->flags & TLS_CONN_SUITEB) && !server &&