narenas
/
opengnsys_ipxe
mirror of https://github.com/ipxe/ipxe.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[tls] Send an empty client certificate chain if we have no certificate 

RFC5246 states that "a client MAY send no certificates if it does not
have an appropriate certificate to send in response to the server's
authentication request".  This use case may arise when the server is
using optional client certificate verification and iPXE has not been
provided with a client certificate to use.

Treat the absence of a suitable client certificate as a non-fatal
condition and send a Certificate message containing no certificates as
permitted by RFC5246.

Reported-by: Alexandre Ravey <alexandre@voilab.ch>
Originally-implemented-by: Alexandre Ravey <alexandre@voilab.ch>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
pull/1437/head
Michael Brown 2025-03-31 14:25:41 +01:00
parent 5818529f39
commit f115cfcf99
1 changed files with 28 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
src/net/tls.c
View File

@ -171,10 +171,6 @@ FILE_LICENCE ( GPL2_OR_LATER );
#define EINFO_EPERM_VERIFY \
__einfo_uniqify ( EINFO_EPERM, 0x02, \
"Handshake verification failed" )
#define EPERM_CLIENT_CERT __einfo_error ( EINFO_EPERM_CLIENT_CERT )
#define EINFO_EPERM_CLIENT_CERT \
__einfo_uniqify ( EINFO_EPERM, 0x03, \
"No suitable client certificate available" )
#define EPERM_RENEG_INSECURE __einfo_error ( EINFO_EPERM_RENEG_INSECURE )
#define EINFO_EPERM_RENEG_INSECURE \
__einfo_uniqify ( EINFO_EPERM, 0x04, \
@ -2464,18 +2460,6 @@ static int tls_new_certificate_request ( struct tls_connection *tls,
x509_chain_put ( tls->client.chain );
tls->client.chain = NULL;
/* Determine client certificate to be sent */
cert = x509_find_key ( NULL, tls->client.key );
if ( ! cert ) {
DBGC ( tls, "TLS %p could not find certificate corresponding "
"to private key\n", tls );
rc = -EPERM_CLIENT_CERT;
goto err_find;
}
x509_get ( cert );
DBGC ( tls, "TLS %p selected client certificate %s\n",
tls, x509_name ( cert ) );
/* Create client certificate chain */
tls->client.chain = x509_alloc_chain();
if ( ! tls->client.chain ) {
@ -2483,26 +2467,41 @@ static int tls_new_certificate_request ( struct tls_connection *tls,
goto err_alloc;
}
/* Determine client certificate to be sent, if any */
cert = x509_find_key ( NULL, tls->client.key );
if ( cert ) {
/* Get temporary reference to certificate */
x509_get ( cert );
DBGC ( tls, "TLS %p selected client certificate %s\n",
tls, x509_name ( cert ) );
/* Append client certificate to chain */
if ( ( rc = x509_append ( tls->client.chain, cert ) ) != 0 )
goto err_append;
/* Append any relevant issuer certificates */
if ( ( rc = x509_auto_append ( tls->client.chain, &certstore ) ) != 0 )
if ( ( rc = x509_auto_append ( tls->client.chain,
&certstore ) ) != 0 )
goto err_auto_append;
} else {
/* Drop local reference to client certificate */
/* Send an empty certificate chain */
DBGC ( tls, "TLS %p could not find certificate corresponding "
"to private key\n", tls );
}
/* Drop local reference (if any) to client certificate */
x509_put ( cert );
return 0;
err_auto_append:
err_append:
x509_put ( cert );
x509_chain_put ( tls->client.chain );
tls->client.chain = NULL;
err_alloc:
x509_put ( cert );
err_find:
return rc;
}
@ -3616,13 +3615,14 @@ static void tls_validator_done ( struct tls_connection *tls, int rc ) {
memcpy ( &tls->server.key, &cert->subject.public_key.raw,
sizeof ( tls->server.key ) );
/* Schedule Client Key Exchange, Change Cipher, and Finished */
/* Schedule transmission of applicable handshake messages */
tls->tx.pending |= ( TLS_TX_CLIENT_KEY_EXCHANGE |
TLS_TX_CHANGE_CIPHER |
TLS_TX_FINISHED );
if ( tls->client.chain ) {
tls->tx.pending |= ( TLS_TX_CERTIFICATE |
TLS_TX_CERTIFICATE_VERIFY );
tls->tx.pending |= TLS_TX_CERTIFICATE;
if ( ! list_empty ( &tls->client.chain->links ) )
tls->tx.pending |= TLS_TX_CERTIFICATE_VERIFY;
}
tls_tx_resume ( tls );