Skip to content

Commit

Permalink
core: basic RawPublicKey support (RFC 7250)
Browse files Browse the repository at this point in the history
  • Loading branch information
@shawnl
shawnl committed Aug 4, 2018
1 parent 99351f0 commit bd883b4
Show file tree
Hide file tree
Showing 4 changed files with 123 additions and 3 deletions.
6 changes: 6 additions & 0 deletions include/picotls.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,7 @@ extern "C" {
#define PTLS_ALERT_BAD_RECORD_MAC 20
#define PTLS_ALERT_HANDSHAKE_FAILURE 40
#define PTLS_ALERT_BAD_CERTIFICATE 42
#define PTLS_ALERT_UNSUPPORTED_CERTIFICATE 43
#define PTLS_ALERT_CERTIFICATE_REVOKED 44
#define PTLS_ALERT_CERTIFICATE_EXPIRED 45
#define PTLS_ALERT_CERTIFICATE_UNKNOWN 46
Expand Down Expand Up @@ -466,6 +467,11 @@ struct st_ptls_context_t {
* to authenticate the client.
*/
unsigned require_client_authentication : 1;
/*
* As for Raw Public Key instead of x509 certificate
*/
unsigned server_raw_public_key : 1;
unsigned client_raw_public_key : 1;
/**
*
*/
Expand Down
3 changes: 3 additions & 0 deletions lib/pembase64.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -367,6 +367,9 @@ int ptls_load_certificates(ptls_context_t *ctx, char const *cert_pem_file)
} else {
ret = ptls_load_pem_objects(cert_pem_file, "CERTIFICATE", ctx->certificates.list, PTLS_MAX_CERTS_IN_CONTEXT,
&ctx->certificates.count);
if (ret != 0)
ret = ptls_load_pem_objects(cert_pem_file, "PUBLIC KEY", ctx->certificates.list, PTLS_MAX_CERTS_IN_CONTEXT,
&ctx->certificates.count);
}

return ret;
Expand Down
109 changes: 107 additions & 2 deletions lib/picotls.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,12 @@
#define PTLS_HANDSHAKE_TYPE_KEY_UPDATE 24
#define PTLS_HANDSHAKE_TYPE_MESSAGE_HASH 254

/* 4.4.2 */
#define PTLS_CERTIFICATE_TYPE_X509 0
#define PTLS_CERTIFICATE_TYPE_X509_BIT 1 << PTLS_CERTIFICATE_TYPE_X509
#define PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY 2
#define PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY_BIT 1 << PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY

#define PTLS_PSK_KE_MODE_PSK 0
#define PTLS_PSK_KE_MODE_PSK_DHE 1

Expand All @@ -66,6 +72,8 @@
#define PTLS_EXTENSION_TYPE_SUPPORTED_GROUPS 10
#define PTLS_EXTENSION_TYPE_SIGNATURE_ALGORITHMS 13
#define PTLS_EXTENSION_TYPE_ALPN 16
#define PTLS_EXTENSION_TYPE_CLIENT_CERTIFICATE_TYPE 19
#define PTLS_EXTENSION_TYPE_SERVER_CERTIFICATE_TYPE 20
#define PTLS_EXTENSION_TYPE_PRE_SHARED_KEY 41
#define PTLS_EXTENSION_TYPE_EARLY_DATA 42
#define PTLS_EXTENSION_TYPE_SUPPORTED_VERSIONS 43
Expand Down Expand Up @@ -217,6 +225,11 @@ struct st_ptls_t {
unsigned is_psk_handshake : 1;
unsigned skip_early_data : 1; /* if early-data is not recognized by the server */
unsigned send_change_cipher_spec : 1;
/* RFC 7250 */
uint8_t client_certificate_types; /* bit field as should have been all along */
uint8_t server_certificate_types; /* bit field as should have been all along */
uint8_t client_certificate_type;
uint8_t server_certificate_type;
/**
* exporter master secret (either 0rtt or 1rtt)
*/
Expand Down Expand Up @@ -1538,6 +1551,22 @@ static int send_client_hello(ptls_t *tls, struct st_ptls_message_emitter_t *emit
}
});
});
/* RFC 7250 RawPublicKey
* TODO make sure all every ptls_key_exchange_algorithm_t implements RawPublicKey
* as per https://tools.ietf.org/html/rfc7250#section-4.1 P 5 and 6
* TODO allow accepting both RAW_PUBLIC_KEY and X509, and then extracting the public key from the certificate
* ptls_buffer_push(sendbuf, 2, PTLS_CERTIFICATE_TYPE_X509, PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY);
*/
if (tls->ctx->client_raw_public_key) {
buffer_push_extension(sendbuf, PTLS_EXTENSION_TYPE_CLIENT_CERTIFICATE_TYPE, {
ptls_buffer_push(sendbuf, 1, PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY);
});
}
if (tls->ctx->server_raw_public_key) {
buffer_push_extension(sendbuf, PTLS_EXTENSION_TYPE_SERVER_CERTIFICATE_TYPE, {
ptls_buffer_push(sendbuf, 1, PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY);
});
}
if (cookie != NULL && cookie->base != NULL) {
buffer_push_extension(sendbuf, PTLS_EXTENSION_TYPE_COOKIE, {
ptls_buffer_push_block(sendbuf, 2, { ptls_buffer_pushv(sendbuf, cookie->base, cookie->len); });
Expand Down Expand Up @@ -1730,12 +1759,43 @@ static int decode_server_hello(ptls_t *tls, struct st_ptls_server_hello_t *sh, c
goto Exit;
}
break;
default:
case PTLS_EXTENSION_TYPE_CLIENT_CERTIFICATE_TYPE:
case PTLS_EXTENSION_TYPE_SERVER_CERTIFICATE_TYPE: {
if (src == end) {
ret = PTLS_ALERT_DECODE_ERROR;
goto Exit;
}
uint8_t certificate_type = *src++;
if (certificate_type > PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY ||
certificate_type == 1/* OpenSSH, forbidden by TLS 1.3 */) {
ret = PTLS_ALERT_UNSUPPORTED_CERTIFICATE;
goto Exit;
}
switch (exttype) {
case PTLS_EXTENSION_TYPE_CLIENT_CERTIFICATE_TYPE:
tls->client_certificate_type = certificate_type;
break;
case PTLS_EXTENSION_TYPE_SERVER_CERTIFICATE_TYPE:
tls->server_certificate_type = certificate_type;
break;
default:
break;
}
break;
} default:
src = end;
break;
}
});

if (((tls->client_certificate_type == PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY) !=
tls->ctx->client_raw_public_key) || /* TODO handle this case by extracting public key from client certificate */
((tls->server_certificate_type == PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY) !=
tls->ctx->server_raw_public_key)) {
ret = PTLS_ALERT_UNSUPPORTED_CERTIFICATE;
goto Exit;
}

if (!is_supported_version(found_version)) {
ret = PTLS_ALERT_ILLEGAL_PARAMETER;
goto Exit;
Expand Down Expand Up @@ -2612,13 +2672,50 @@ static int decode_client_hello(ptls_t *tls, struct st_ptls_client_hello_t *ch, c
case PTLS_EXTENSION_TYPE_STATUS_REQUEST:
ch->status_request = 1;
break;
default:
case PTLS_EXTENSION_TYPE_CLIENT_CERTIFICATE_TYPE:
case PTLS_EXTENSION_TYPE_SERVER_CERTIFICATE_TYPE: {
if (src == end) {
ret = PTLS_ALERT_DECODE_ERROR;
goto Exit;
}
uint8_t length = *src++;
if (end - src != length) {
ret = PTLS_ALERT_DECODE_ERROR;
goto Exit;
}
for (;src != end; ++src) {
uint8_t certificate_type = *src;
if (certificate_type > PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY ||
certificate_type == 1/* OpenSSH, forbidden by TLS 1.3 */) {
ret = PTLS_ALERT_UNSUPPORTED_CERTIFICATE;
goto Exit;
}
if (exttype == PTLS_EXTENSION_TYPE_CLIENT_CERTIFICATE_TYPE) {
tls->client_certificate_types |= (1 << certificate_type);
} else if (exttype == PTLS_EXTENSION_TYPE_SERVER_CERTIFICATE_TYPE) {
tls->server_certificate_types |= (1 << certificate_type);
}
}
break;
} default:
handle_unknown_extension(tls, properties, exttype, src, end, ch->unknown_extensions);
break;
}
src = end;
});

if ((tls->client_certificate_types & PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY_BIT) &&
!tls->ctx->client_raw_public_key ) {
ret = PTLS_ALERT_UNSUPPORTED_CERTIFICATE;
goto Exit;
}
/* TODO handle this case by extracting public key from certificate */
if ((tls->server_certificate_types & PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY_BIT) &&
!tls->ctx->server_raw_public_key) {
ret = PTLS_ALERT_UNSUPPORTED_CERTIFICATE;
goto Exit;
}

/* check if client hello make sense */
if (is_supported_version(ch->selected_version)) {
if (!(ch->compression_methods.count == 1 && ch->compression_methods.ids[0] == 0)) {
Expand Down Expand Up @@ -3072,6 +3169,14 @@ static int server_handle_hello(ptls_t *tls, struct st_ptls_message_emitter_t *em
buffer_push_extension(sendbuf, PTLS_EXTENSION_TYPE_PRE_SHARED_KEY,
{ ptls_buffer_push16(sendbuf, (uint16_t)psk_index); });
}
if (tls->ctx->server_raw_public_key)
buffer_push_extension(sendbuf, PTLS_EXTENSION_TYPE_SERVER_CERTIFICATE_TYPE, {
ptls_buffer_push(sendbuf, PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY);
});
if (tls->ctx->client_raw_public_key)
buffer_push_extension(sendbuf, PTLS_EXTENSION_TYPE_CLIENT_CERTIFICATE_TYPE, {
ptls_buffer_push(sendbuf, PTLS_CERTIFICATE_TYPE_RAW_PUBLIC_KEY);
});
});
if ((ret = push_change_cipher_spec(tls, emitter->buf)) != 0)
goto Exit;
Expand Down
8 changes: 7 additions & 1 deletion t/cli-minicrypto.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ int main(int argc, char **argv)
socklen_t salen;
int family = 0;

while ((ch = getopt(argc, argv, "46aC:c:i:k:nN:es:Sl:vh:jJ")) != -1) {
while ((ch = getopt(argc, argv, "46aC:c:i:k:nN:es:Sl:vh:jJrR")) != -1) {
switch (ch) {
case '4':
family = AF_INET;
Expand All @@ -337,6 +337,12 @@ int main(int argc, char **argv)
load_certificate_chain(&ctx, optarg);
is_server = ch == 'c';
break;
case 'r':
ctx.server_raw_public_key = 1;
break;
case 'R':
ctx.client_raw_public_key = 1;
break;
case 'i':
file = optarg;
break;
Expand Down

0 comments on commit bd883b4

Please sign in to comment.