blob: 8e994e5819e892e47cd27d8b91dfef0ad67a4b63 [file] [log] [blame]
Steven Valdez143e8b32016-07-11 13:19:03 -04001/* Copyright (c) 2016, Google Inc.
2 *
3 * Permission to use, copy, modify, and/or distribute this software for any
4 * purpose with or without fee is hereby granted, provided that the above
5 * copyright notice and this permission notice appear in all copies.
6 *
7 * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
8 * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
9 * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY
10 * SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
11 * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION
12 * OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN
13 * CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. */
14
15#include <openssl/ssl.h>
16
17#include <assert.h>
David Benjamin17b30832017-01-28 14:00:32 -050018#include <limits.h>
Steven Valdez143e8b32016-07-11 13:19:03 -040019#include <string.h>
20
21#include <openssl/bytestring.h>
22#include <openssl/digest.h>
23#include <openssl/err.h>
24#include <openssl/mem.h>
25#include <openssl/stack.h>
26#include <openssl/x509.h>
27
David Benjaminffb11072016-11-13 10:32:10 +090028#include "../crypto/internal.h"
Steven Valdez143e8b32016-07-11 13:19:03 -040029#include "internal.h"
30
31
32enum client_hs_state_t {
Steven Valdez5440fe02016-07-18 12:40:30 -040033 state_process_hello_retry_request = 0,
34 state_send_second_client_hello,
Steven Valdez5440fe02016-07-18 12:40:30 -040035 state_process_server_hello,
Steven Valdez143e8b32016-07-11 13:19:03 -040036 state_process_encrypted_extensions,
37 state_process_certificate_request,
38 state_process_server_certificate,
39 state_process_server_certificate_verify,
40 state_process_server_finished,
Steven Valdez143e8b32016-07-11 13:19:03 -040041 state_send_client_certificate,
42 state_send_client_certificate_verify,
43 state_complete_client_certificate_verify,
David Benjamin81b7bc32017-01-12 19:44:57 -050044 state_complete_second_flight,
Steven Valdez143e8b32016-07-11 13:19:03 -040045 state_done,
46};
47
David Benjamin8f820b42016-11-30 11:24:40 -050048static const uint8_t kZeroes[EVP_MAX_MD_SIZE] = {0};
49
David Benjaminc3c88822016-11-14 10:32:04 +090050static enum ssl_hs_wait_t do_process_hello_retry_request(SSL_HANDSHAKE *hs) {
51 SSL *const ssl = hs->ssl;
Steven Valdez5440fe02016-07-18 12:40:30 -040052 if (ssl->s3->tmp.message_type != SSL3_MT_HELLO_RETRY_REQUEST) {
David Benjamin3977f302016-12-11 13:30:41 -050053 hs->tls13_state = state_process_server_hello;
Steven Valdez5440fe02016-07-18 12:40:30 -040054 return ssl_hs_ok;
55 }
56
57 CBS cbs, extensions;
David Benjamin3baa6e12016-10-07 21:10:38 -040058 uint16_t server_wire_version;
Steven Valdez5440fe02016-07-18 12:40:30 -040059 CBS_init(&cbs, ssl->init_msg, ssl->init_num);
60 if (!CBS_get_u16(&cbs, &server_wire_version) ||
Steven Valdez5440fe02016-07-18 12:40:30 -040061 !CBS_get_u16_length_prefixed(&cbs, &extensions) ||
David Benjamin3baa6e12016-10-07 21:10:38 -040062 /* HelloRetryRequest may not be empty. */
63 CBS_len(&extensions) == 0 ||
Steven Valdez5440fe02016-07-18 12:40:30 -040064 CBS_len(&cbs) != 0) {
David Benjamin3baa6e12016-10-07 21:10:38 -040065 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
Steven Valdez5440fe02016-07-18 12:40:30 -040066 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
67 return ssl_hs_error;
68 }
69
David Benjaminffb11072016-11-13 10:32:10 +090070 int have_cookie, have_key_share;
71 CBS cookie, key_share;
72 const SSL_EXTENSION_TYPE ext_types[] = {
73 {TLSEXT_TYPE_key_share, &have_key_share, &key_share},
74 {TLSEXT_TYPE_cookie, &have_cookie, &cookie},
75 };
76
Adam Langleyc68e5b92017-02-08 13:33:15 -080077 uint8_t alert = SSL_AD_DECODE_ERROR;
David Benjaminffb11072016-11-13 10:32:10 +090078 if (!ssl_parse_extensions(&extensions, &alert, ext_types,
Steven Valdez08b65f42016-12-07 15:29:45 -050079 OPENSSL_ARRAY_SIZE(ext_types),
80 0 /* reject unknown */)) {
David Benjaminffb11072016-11-13 10:32:10 +090081 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
82 return ssl_hs_error;
83 }
84
85 if (have_cookie) {
86 CBS cookie_value;
87 if (!CBS_get_u16_length_prefixed(&cookie, &cookie_value) ||
88 CBS_len(&cookie_value) == 0 ||
89 CBS_len(&cookie) != 0) {
David Benjamin3baa6e12016-10-07 21:10:38 -040090 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
91 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
92 return ssl_hs_error;
93 }
Steven Valdez5440fe02016-07-18 12:40:30 -040094
David Benjaminffb11072016-11-13 10:32:10 +090095 if (!CBS_stow(&cookie_value, &hs->cookie, &hs->cookie_len)) {
96 return ssl_hs_error;
Steven Valdez5440fe02016-07-18 12:40:30 -040097 }
98 }
99
David Benjaminffb11072016-11-13 10:32:10 +0900100 if (have_key_share) {
101 uint16_t group_id;
102 if (!CBS_get_u16(&key_share, &group_id) || CBS_len(&key_share) != 0) {
103 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
104 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
105 return ssl_hs_error;
106 }
107
108 /* The group must be supported. */
109 const uint16_t *groups;
110 size_t groups_len;
111 tls1_get_grouplist(ssl, &groups, &groups_len);
112 int found = 0;
113 for (size_t i = 0; i < groups_len; i++) {
114 if (groups[i] == group_id) {
115 found = 1;
116 break;
117 }
118 }
119
120 if (!found) {
121 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER);
122 OPENSSL_PUT_ERROR(SSL, SSL_R_WRONG_CURVE);
123 return ssl_hs_error;
124 }
125
126 /* Check that the HelloRetryRequest does not request the key share that
127 * was provided in the initial ClientHello. */
128 if (SSL_ECDH_CTX_get_id(&hs->ecdh_ctx) == group_id) {
129 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER);
130 OPENSSL_PUT_ERROR(SSL, SSL_R_WRONG_CURVE);
131 return ssl_hs_error;
132 }
133
134 SSL_ECDH_CTX_cleanup(&hs->ecdh_ctx);
135 hs->retry_group = group_id;
136 }
137
Steven Valdez908ac192017-01-12 13:17:07 -0500138 if (!ssl_hash_current_message(hs)) {
David Benjaminf71036e2017-01-21 14:49:39 -0500139 return ssl_hs_error;
140 }
141
David Benjamin3baa6e12016-10-07 21:10:38 -0400142 hs->received_hello_retry_request = 1;
David Benjamin3977f302016-12-11 13:30:41 -0500143 hs->tls13_state = state_send_second_client_hello;
Steven Valdez5440fe02016-07-18 12:40:30 -0400144 return ssl_hs_ok;
145}
146
David Benjaminc3c88822016-11-14 10:32:04 +0900147static enum ssl_hs_wait_t do_send_second_client_hello(SSL_HANDSHAKE *hs) {
David Benjamin8c880a22016-12-03 02:20:34 -0500148 if (!ssl_write_client_hello(hs)) {
Steven Valdez5440fe02016-07-18 12:40:30 -0400149 return ssl_hs_error;
150 }
151
David Benjamin3977f302016-12-11 13:30:41 -0500152 hs->tls13_state = state_process_server_hello;
Steven Valdez5440fe02016-07-18 12:40:30 -0400153 return ssl_hs_flush_and_read_message;
154}
155
David Benjaminc3c88822016-11-14 10:32:04 +0900156static enum ssl_hs_wait_t do_process_server_hello(SSL_HANDSHAKE *hs) {
157 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500158 if (!ssl_check_message_type(ssl, SSL3_MT_SERVER_HELLO)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400159 return ssl_hs_error;
160 }
161
162 CBS cbs, server_random, extensions;
163 uint16_t server_wire_version;
164 uint16_t cipher_suite;
165 CBS_init(&cbs, ssl->init_msg, ssl->init_num);
166 if (!CBS_get_u16(&cbs, &server_wire_version) ||
167 !CBS_get_bytes(&cbs, &server_random, SSL3_RANDOM_SIZE) ||
168 !CBS_get_u16(&cbs, &cipher_suite) ||
169 !CBS_get_u16_length_prefixed(&cbs, &extensions) ||
170 CBS_len(&cbs) != 0) {
171 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
Steven Valdez5440fe02016-07-18 12:40:30 -0400172 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
173 return ssl_hs_error;
174 }
175
176 if (server_wire_version != ssl->version) {
177 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
178 OPENSSL_PUT_ERROR(SSL, SSL_R_WRONG_VERSION_NUMBER);
Steven Valdez143e8b32016-07-11 13:19:03 -0400179 return ssl_hs_error;
180 }
181
Steven Valdez803c77a2016-09-06 14:13:43 -0400182 assert(ssl->s3->have_version);
David Benjamin17cf2cb2016-12-13 01:07:13 -0500183 OPENSSL_memcpy(ssl->s3->server_random, CBS_data(&server_random),
184 SSL3_RANDOM_SIZE);
Steven Valdez803c77a2016-09-06 14:13:43 -0400185
186 const SSL_CIPHER *cipher = SSL_get_cipher_by_value(cipher_suite);
187 if (cipher == NULL) {
188 OPENSSL_PUT_ERROR(SSL, SSL_R_UNKNOWN_CIPHER_RETURNED);
189 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER);
190 return ssl_hs_error;
191 }
192
David Benjaminabbbee12016-10-31 19:20:42 -0400193 /* Check if the cipher is a TLS 1.3 cipher. */
194 if (SSL_CIPHER_get_min_version(cipher) > ssl3_protocol_version(ssl) ||
195 SSL_CIPHER_get_max_version(cipher) < ssl3_protocol_version(ssl)) {
Steven Valdez803c77a2016-09-06 14:13:43 -0400196 OPENSSL_PUT_ERROR(SSL, SSL_R_WRONG_CIPHER_RETURNED);
197 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER);
198 return ssl_hs_error;
199 }
200
Steven Valdez143e8b32016-07-11 13:19:03 -0400201 /* Parse out the extensions. */
David Benjamin6f600d62016-12-21 16:06:54 -0500202 int have_key_share = 0, have_pre_shared_key = 0, have_short_header = 0;
203 CBS key_share, pre_shared_key, short_header;
David Benjaminffb11072016-11-13 10:32:10 +0900204 const SSL_EXTENSION_TYPE ext_types[] = {
205 {TLSEXT_TYPE_key_share, &have_key_share, &key_share},
206 {TLSEXT_TYPE_pre_shared_key, &have_pre_shared_key, &pre_shared_key},
David Benjamin6f600d62016-12-21 16:06:54 -0500207 {TLSEXT_TYPE_short_header, &have_short_header, &short_header},
David Benjaminffb11072016-11-13 10:32:10 +0900208 };
Steven Valdez143e8b32016-07-11 13:19:03 -0400209
Adam Langleyc68e5b92017-02-08 13:33:15 -0800210 uint8_t alert = SSL_AD_DECODE_ERROR;
David Benjaminffb11072016-11-13 10:32:10 +0900211 if (!ssl_parse_extensions(&extensions, &alert, ext_types,
Steven Valdez08b65f42016-12-07 15:29:45 -0500212 OPENSSL_ARRAY_SIZE(ext_types),
213 0 /* reject unknown */)) {
David Benjaminffb11072016-11-13 10:32:10 +0900214 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
215 return ssl_hs_error;
Steven Valdez143e8b32016-07-11 13:19:03 -0400216 }
217
David Benjaminffb11072016-11-13 10:32:10 +0900218 alert = SSL_AD_DECODE_ERROR;
Steven Valdez4aa154e2016-07-29 14:32:55 -0400219 if (have_pre_shared_key) {
220 if (ssl->session == NULL) {
221 OPENSSL_PUT_ERROR(SSL, SSL_R_UNEXPECTED_EXTENSION);
222 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_UNSUPPORTED_EXTENSION);
223 return ssl_hs_error;
224 }
225
David Benjamin8baf9632016-11-17 17:11:16 +0900226 if (!ssl_ext_pre_shared_key_parse_serverhello(hs, &alert,
Steven Valdez4aa154e2016-07-29 14:32:55 -0400227 &pre_shared_key)) {
228 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
229 return ssl_hs_error;
230 }
231
232 if (ssl->session->ssl_version != ssl->version) {
233 OPENSSL_PUT_ERROR(SSL, SSL_R_OLD_SESSION_VERSION_NOT_RETURNED);
234 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER);
235 return ssl_hs_error;
236 }
237
David Benjamine1cc35e2016-11-16 16:25:58 +0900238 if (ssl->session->cipher->algorithm_prf != cipher->algorithm_prf) {
239 OPENSSL_PUT_ERROR(SSL, SSL_R_OLD_SESSION_PRF_HASH_MISMATCH);
Steven Valdez803c77a2016-09-06 14:13:43 -0400240 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER);
241 return ssl_hs_error;
242 }
243
Steven Valdez4aa154e2016-07-29 14:32:55 -0400244 if (!ssl_session_is_context_valid(ssl, ssl->session)) {
245 /* This is actually a client application bug. */
246 OPENSSL_PUT_ERROR(SSL,
247 SSL_R_ATTEMPT_TO_REUSE_SESSION_IN_DIFFERENT_CONTEXT);
248 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER);
249 return ssl_hs_error;
250 }
251
252 ssl->s3->session_reused = 1;
253 /* Only authentication information carries over in TLS 1.3. */
David Benjamin45738dd2017-02-09 20:01:26 -0500254 hs->new_session = SSL_SESSION_dup(ssl->session, SSL_SESSION_DUP_AUTH_ONLY);
255 if (hs->new_session == NULL) {
Steven Valdez4aa154e2016-07-29 14:32:55 -0400256 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR);
257 return ssl_hs_error;
258 }
David Benjamin4d0be242016-09-01 01:10:07 -0400259 ssl_set_session(ssl, NULL);
David Benjamin17b30832017-01-28 14:00:32 -0500260
261 /* Resumption incorporates fresh key material, so refresh the timeout. */
David Benjamin45738dd2017-02-09 20:01:26 -0500262 ssl_session_renew_timeout(ssl, hs->new_session,
Alessandro Ghedini33fe4a02017-02-06 13:40:10 +0000263 ssl->initial_ctx->session_psk_dhe_timeout);
David Benjaminf3c8f8d2016-11-17 17:20:47 +0900264 } else if (!ssl_get_new_session(hs, 0)) {
Steven Valdez803c77a2016-09-06 14:13:43 -0400265 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR);
Steven Valdez143e8b32016-07-11 13:19:03 -0400266 return ssl_hs_error;
267 }
268
David Benjamin45738dd2017-02-09 20:01:26 -0500269 hs->new_session->cipher = cipher;
270 hs->new_cipher = cipher;
Steven Valdez143e8b32016-07-11 13:19:03 -0400271
Steven Valdez27a9e6a2017-02-14 13:20:40 -0500272 /* Store the initial negotiated ALPN in the session. */
273 if (ssl->s3->alpn_selected != NULL) {
David Benjamin45738dd2017-02-09 20:01:26 -0500274 hs->new_session->early_alpn =
Steven Valdez27a9e6a2017-02-14 13:20:40 -0500275 BUF_memdup(ssl->s3->alpn_selected, ssl->s3->alpn_selected_len);
David Benjamin45738dd2017-02-09 20:01:26 -0500276 if (hs->new_session->early_alpn == NULL) {
Steven Valdez27a9e6a2017-02-14 13:20:40 -0500277 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR);
278 return ssl_hs_error;
279 }
David Benjamin45738dd2017-02-09 20:01:26 -0500280 hs->new_session->early_alpn_len = ssl->s3->alpn_selected_len;
Steven Valdez27a9e6a2017-02-14 13:20:40 -0500281 }
282
Steven Valdez143e8b32016-07-11 13:19:03 -0400283 /* The PRF hash is now known. Set up the key schedule. */
David Benjamin6e4fc332016-11-17 16:43:08 +0900284 if (!tls13_init_key_schedule(hs)) {
David Benjamin8f820b42016-11-30 11:24:40 -0500285 return ssl_hs_error;
Steven Valdez143e8b32016-07-11 13:19:03 -0400286 }
287
David Benjamin8f820b42016-11-30 11:24:40 -0500288 /* Incorporate the PSK into the running secret. */
289 if (ssl->s3->session_reused) {
David Benjamin45738dd2017-02-09 20:01:26 -0500290 if (!tls13_advance_key_schedule(hs, hs->new_session->master_key,
291 hs->new_session->master_key_length)) {
David Benjamin8f820b42016-11-30 11:24:40 -0500292 return ssl_hs_error;
293 }
Steven Valdez908ac192017-01-12 13:17:07 -0500294 } else if (!tls13_advance_key_schedule(hs, kZeroes, hs->hash_len)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400295 return ssl_hs_error;
296 }
297
David Benjamindb5bd722016-12-08 18:21:27 -0500298 if (!have_key_share) {
299 /* We do not support psk_ke and thus always require a key share. */
300 OPENSSL_PUT_ERROR(SSL, SSL_R_MISSING_KEY_SHARE);
301 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_MISSING_EXTENSION);
302 return ssl_hs_error;
303 }
304
Steven Valdez143e8b32016-07-11 13:19:03 -0400305 /* Resolve ECDHE and incorporate it into the secret. */
Steven Valdez803c77a2016-09-06 14:13:43 -0400306 uint8_t *dhe_secret;
307 size_t dhe_secret_len;
Adam Langleyc68e5b92017-02-08 13:33:15 -0800308 alert = SSL_AD_DECODE_ERROR;
David Benjamin8baf9632016-11-17 17:11:16 +0900309 if (!ssl_ext_key_share_parse_serverhello(hs, &dhe_secret, &dhe_secret_len,
Steven Valdez803c77a2016-09-06 14:13:43 -0400310 &alert, &key_share)) {
311 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
312 return ssl_hs_error;
313 }
Steven Valdez143e8b32016-07-11 13:19:03 -0400314
David Benjamin6e4fc332016-11-17 16:43:08 +0900315 if (!tls13_advance_key_schedule(hs, dhe_secret, dhe_secret_len)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400316 OPENSSL_free(dhe_secret);
Steven Valdez803c77a2016-09-06 14:13:43 -0400317 return ssl_hs_error;
318 }
319 OPENSSL_free(dhe_secret);
320
David Benjamin6f600d62016-12-21 16:06:54 -0500321 /* Negotiate short record headers. */
322 if (have_short_header) {
323 if (CBS_len(&short_header) != 0) {
324 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
325 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
326 return ssl_hs_error;
327 }
328
329 if (!ssl->ctx->short_header_enabled) {
330 OPENSSL_PUT_ERROR(SSL, SSL_R_UNEXPECTED_EXTENSION);
331 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_UNSUPPORTED_EXTENSION);
332 return ssl_hs_error;
333 }
334
335 ssl->s3->short_header = 1;
336 }
337
Steven Valdez908ac192017-01-12 13:17:07 -0500338 if (!ssl_hash_current_message(hs) ||
David Benjaminf71036e2017-01-21 14:49:39 -0500339 !tls13_derive_handshake_secrets(hs) ||
Steven Valdez4cb84942016-12-16 11:29:28 -0500340 !tls13_set_traffic_key(ssl, evp_aead_open, hs->server_handshake_secret,
341 hs->hash_len) ||
342 !tls13_set_traffic_key(ssl, evp_aead_seal, hs->client_handshake_secret,
343 hs->hash_len)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400344 return ssl_hs_error;
345 }
346
David Benjamin3977f302016-12-11 13:30:41 -0500347 hs->tls13_state = state_process_encrypted_extensions;
Steven Valdez143e8b32016-07-11 13:19:03 -0400348 return ssl_hs_read_message;
349}
350
David Benjaminc3c88822016-11-14 10:32:04 +0900351static enum ssl_hs_wait_t do_process_encrypted_extensions(SSL_HANDSHAKE *hs) {
352 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500353 if (!ssl_check_message_type(ssl, SSL3_MT_ENCRYPTED_EXTENSIONS)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400354 return ssl_hs_error;
355 }
356
357 CBS cbs;
358 CBS_init(&cbs, ssl->init_msg, ssl->init_num);
David Benjamin8c880a22016-12-03 02:20:34 -0500359 if (!ssl_parse_serverhello_tlsext(hs, &cbs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400360 OPENSSL_PUT_ERROR(SSL, SSL_R_PARSE_TLSEXT);
361 return ssl_hs_error;
362 }
363 if (CBS_len(&cbs) != 0) {
364 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
365 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
366 return ssl_hs_error;
367 }
368
Steven Valdez908ac192017-01-12 13:17:07 -0500369 if (!ssl_hash_current_message(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400370 return ssl_hs_error;
371 }
372
David Benjamin3977f302016-12-11 13:30:41 -0500373 hs->tls13_state = state_process_certificate_request;
Steven Valdez143e8b32016-07-11 13:19:03 -0400374 return ssl_hs_read_message;
375}
376
David Benjaminc3c88822016-11-14 10:32:04 +0900377static enum ssl_hs_wait_t do_process_certificate_request(SSL_HANDSHAKE *hs) {
378 SSL *const ssl = hs->ssl;
Steven Valdez803c77a2016-09-06 14:13:43 -0400379 /* CertificateRequest may only be sent in non-resumption handshakes. */
380 if (ssl->s3->session_reused) {
David Benjamin3977f302016-12-11 13:30:41 -0500381 hs->tls13_state = state_process_server_finished;
Steven Valdez143e8b32016-07-11 13:19:03 -0400382 return ssl_hs_ok;
383 }
384
385 /* CertificateRequest is optional. */
386 if (ssl->s3->tmp.message_type != SSL3_MT_CERTIFICATE_REQUEST) {
David Benjamin3977f302016-12-11 13:30:41 -0500387 hs->tls13_state = state_process_server_certificate;
Steven Valdez143e8b32016-07-11 13:19:03 -0400388 return ssl_hs_ok;
389 }
390
391 CBS cbs, context, supported_signature_algorithms;
392 CBS_init(&cbs, ssl->init_msg, ssl->init_num);
393 if (!CBS_get_u8_length_prefixed(&cbs, &context) ||
David Benjamin8a8349b2016-08-18 02:32:23 -0400394 /* The request context is always empty during the handshake. */
395 CBS_len(&context) != 0 ||
Steven Valdez143e8b32016-07-11 13:19:03 -0400396 !CBS_get_u16_length_prefixed(&cbs, &supported_signature_algorithms) ||
David Benjamin48901652016-08-01 12:12:47 -0400397 CBS_len(&supported_signature_algorithms) == 0 ||
David Benjaminf3c8f8d2016-11-17 17:20:47 +0900398 !tls1_parse_peer_sigalgs(hs, &supported_signature_algorithms)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400399 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
400 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
401 return ssl_hs_error;
402 }
403
Adam Langleyc68e5b92017-02-08 13:33:15 -0800404 uint8_t alert = SSL_AD_DECODE_ERROR;
Steven Valdez143e8b32016-07-11 13:19:03 -0400405 STACK_OF(X509_NAME) *ca_sk = ssl_parse_client_CA_list(ssl, &alert, &cbs);
406 if (ca_sk == NULL) {
407 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
408 return ssl_hs_error;
409 }
410
411 /* Ignore extensions. */
412 CBS extensions;
413 if (!CBS_get_u16_length_prefixed(&cbs, &extensions) ||
414 CBS_len(&cbs) != 0) {
415 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
David Benjamin639846e2016-09-09 11:41:18 -0400416 sk_X509_NAME_pop_free(ca_sk, X509_NAME_free);
Steven Valdez143e8b32016-07-11 13:19:03 -0400417 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
418 return ssl_hs_error;
419 }
420
David Benjaminc3c88822016-11-14 10:32:04 +0900421 hs->cert_request = 1;
422 sk_X509_NAME_pop_free(hs->ca_names, X509_NAME_free);
423 hs->ca_names = ca_sk;
Steven Valdez143e8b32016-07-11 13:19:03 -0400424
Steven Valdez908ac192017-01-12 13:17:07 -0500425 if (!ssl_hash_current_message(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400426 return ssl_hs_error;
427 }
428
David Benjamin3977f302016-12-11 13:30:41 -0500429 hs->tls13_state = state_process_server_certificate;
Steven Valdez143e8b32016-07-11 13:19:03 -0400430 return ssl_hs_read_message;
431}
432
David Benjaminc3c88822016-11-14 10:32:04 +0900433static enum ssl_hs_wait_t do_process_server_certificate(SSL_HANDSHAKE *hs) {
434 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500435 if (!ssl_check_message_type(ssl, SSL3_MT_CERTIFICATE) ||
Adam Langley0c294252016-12-12 11:46:09 -0800436 !tls13_process_certificate(hs, 0 /* certificate required */) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500437 !ssl_hash_current_message(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400438 return ssl_hs_error;
439 }
440
David Benjamin3977f302016-12-11 13:30:41 -0500441 hs->tls13_state = state_process_server_certificate_verify;
Steven Valdez143e8b32016-07-11 13:19:03 -0400442 return ssl_hs_read_message;
443}
444
445static enum ssl_hs_wait_t do_process_server_certificate_verify(
David Benjaminc3c88822016-11-14 10:32:04 +0900446 SSL_HANDSHAKE *hs) {
447 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500448 if (!ssl_check_message_type(ssl, SSL3_MT_CERTIFICATE_VERIFY) ||
Adam Langley0c294252016-12-12 11:46:09 -0800449 !tls13_process_certificate_verify(hs) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500450 !ssl_hash_current_message(hs)) {
David Benjamin6929f272016-11-16 19:10:08 +0900451 return ssl_hs_error;
Steven Valdez143e8b32016-07-11 13:19:03 -0400452 }
453
David Benjamin3977f302016-12-11 13:30:41 -0500454 hs->tls13_state = state_process_server_finished;
Steven Valdez143e8b32016-07-11 13:19:03 -0400455 return ssl_hs_read_message;
456}
457
David Benjaminc3c88822016-11-14 10:32:04 +0900458static enum ssl_hs_wait_t do_process_server_finished(SSL_HANDSHAKE *hs) {
459 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500460 if (!ssl_check_message_type(ssl, SSL3_MT_FINISHED) ||
David Benjamin6e4fc332016-11-17 16:43:08 +0900461 !tls13_process_finished(hs) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500462 !ssl_hash_current_message(hs) ||
Steven Valdez143e8b32016-07-11 13:19:03 -0400463 /* Update the secret to the master secret and derive traffic keys. */
David Benjamin6e4fc332016-11-17 16:43:08 +0900464 !tls13_advance_key_schedule(hs, kZeroes, hs->hash_len) ||
465 !tls13_derive_application_secrets(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400466 return ssl_hs_error;
467 }
468
David Benjamin613fe3b2016-07-22 17:39:29 +0200469 ssl->method->received_flight(ssl);
David Benjamin5edfc8c2016-12-10 15:46:58 -0500470 hs->tls13_state = state_send_client_certificate;
Steven Valdez143e8b32016-07-11 13:19:03 -0400471 return ssl_hs_ok;
472}
473
David Benjamin5edfc8c2016-12-10 15:46:58 -0500474static enum ssl_hs_wait_t do_send_client_certificate(SSL_HANDSHAKE *hs) {
David Benjaminc3c88822016-11-14 10:32:04 +0900475 SSL *const ssl = hs->ssl;
Steven Valdez143e8b32016-07-11 13:19:03 -0400476 /* The peer didn't request a certificate. */
David Benjaminc3c88822016-11-14 10:32:04 +0900477 if (!hs->cert_request) {
David Benjamin81b7bc32017-01-12 19:44:57 -0500478 hs->tls13_state = state_complete_second_flight;
Steven Valdez143e8b32016-07-11 13:19:03 -0400479 return ssl_hs_ok;
480 }
481
482 /* Call cert_cb to update the certificate. */
483 if (ssl->cert->cert_cb != NULL) {
484 int rv = ssl->cert->cert_cb(ssl, ssl->cert->cert_cb_arg);
485 if (rv == 0) {
486 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR);
487 OPENSSL_PUT_ERROR(SSL, SSL_R_CERT_CB_ERROR);
488 return ssl_hs_error;
489 }
490 if (rv < 0) {
David Benjamin3977f302016-12-11 13:30:41 -0500491 hs->tls13_state = state_send_client_certificate;
Steven Valdez143e8b32016-07-11 13:19:03 -0400492 return ssl_hs_x509_lookup;
493 }
Steven Valdez143e8b32016-07-11 13:19:03 -0400494 }
495
David Benjamin650aa1c2016-12-20 18:55:16 -0500496 if (!ssl_auto_chain_if_needed(ssl) ||
David Benjamin0f24bed2017-01-12 19:46:50 -0500497 !tls13_add_certificate(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400498 return ssl_hs_error;
499 }
500
David Benjamin3977f302016-12-11 13:30:41 -0500501 hs->tls13_state = state_send_client_certificate_verify;
David Benjamin25ac2512017-01-12 19:31:28 -0500502 return ssl_hs_ok;
Steven Valdez143e8b32016-07-11 13:19:03 -0400503}
504
David Benjaminc3c88822016-11-14 10:32:04 +0900505static enum ssl_hs_wait_t do_send_client_certificate_verify(SSL_HANDSHAKE *hs,
Steven Valdez143e8b32016-07-11 13:19:03 -0400506 int is_first_run) {
David Benjaminc3c88822016-11-14 10:32:04 +0900507 SSL *const ssl = hs->ssl;
Steven Valdez143e8b32016-07-11 13:19:03 -0400508 /* Don't send CertificateVerify if there is no certificate. */
509 if (!ssl_has_certificate(ssl)) {
David Benjamin81b7bc32017-01-12 19:44:57 -0500510 hs->tls13_state = state_complete_second_flight;
Steven Valdez143e8b32016-07-11 13:19:03 -0400511 return ssl_hs_ok;
512 }
513
David Benjamin0f24bed2017-01-12 19:46:50 -0500514 switch (tls13_add_certificate_verify(hs, is_first_run)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400515 case ssl_private_key_success:
David Benjamin81b7bc32017-01-12 19:44:57 -0500516 hs->tls13_state = state_complete_second_flight;
David Benjamin25ac2512017-01-12 19:31:28 -0500517 return ssl_hs_ok;
Steven Valdez143e8b32016-07-11 13:19:03 -0400518
519 case ssl_private_key_retry:
David Benjamin3977f302016-12-11 13:30:41 -0500520 hs->tls13_state = state_complete_client_certificate_verify;
Steven Valdez143e8b32016-07-11 13:19:03 -0400521 return ssl_hs_private_key_operation;
522
523 case ssl_private_key_failure:
524 return ssl_hs_error;
525 }
526
527 assert(0);
528 return ssl_hs_error;
529}
530
David Benjamin81b7bc32017-01-12 19:44:57 -0500531static enum ssl_hs_wait_t do_complete_second_flight(SSL_HANDSHAKE *hs) {
David Benjaminc3c88822016-11-14 10:32:04 +0900532 SSL *const ssl = hs->ssl;
David Benjamin81b7bc32017-01-12 19:44:57 -0500533
534 /* Send a Channel ID assertion if necessary. */
535 if (ssl->s3->tlsext_channel_id_valid) {
536 if (!ssl_do_channel_id_callback(ssl)) {
537 hs->tls13_state = state_complete_second_flight;
538 return ssl_hs_error;
539 }
540
541 if (ssl->tlsext_channel_id_private == NULL) {
542 return ssl_hs_channel_id_lookup;
543 }
544
545 CBB cbb, body;
546 if (!ssl->method->init_message(ssl, &cbb, &body, SSL3_MT_CHANNEL_ID) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500547 !tls1_write_channel_id(hs, &body) ||
David Benjamin81b7bc32017-01-12 19:44:57 -0500548 !ssl_add_message_cbb(ssl, &cbb)) {
549 CBB_cleanup(&cbb);
550 return ssl_hs_error;
551 }
Nick Harper60a85cb2016-09-23 16:25:11 -0700552 }
553
David Benjamin81b7bc32017-01-12 19:44:57 -0500554 /* Send a Finished message. */
David Benjamin0f24bed2017-01-12 19:46:50 -0500555 if (!tls13_add_finished(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400556 return ssl_hs_error;
557 }
558
David Benjamin81b7bc32017-01-12 19:44:57 -0500559 /* Derive the final keys and enable them. */
Steven Valdeza833c352016-11-01 13:39:36 -0400560 if (!tls13_set_traffic_key(ssl, evp_aead_open, hs->server_traffic_secret_0,
561 hs->hash_len) ||
562 !tls13_set_traffic_key(ssl, evp_aead_seal, hs->client_traffic_secret_0,
563 hs->hash_len) ||
David Benjamin6e4fc332016-11-17 16:43:08 +0900564 !tls13_derive_resumption_secret(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400565 return ssl_hs_error;
566 }
567
David Benjamin3977f302016-12-11 13:30:41 -0500568 hs->tls13_state = state_done;
Steven Valdez143e8b32016-07-11 13:19:03 -0400569 return ssl_hs_flush;
570}
571
David Benjaminc3c88822016-11-14 10:32:04 +0900572enum ssl_hs_wait_t tls13_client_handshake(SSL_HANDSHAKE *hs) {
David Benjamin3977f302016-12-11 13:30:41 -0500573 while (hs->tls13_state != state_done) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400574 enum ssl_hs_wait_t ret = ssl_hs_error;
David Benjamin3977f302016-12-11 13:30:41 -0500575 enum client_hs_state_t state = hs->tls13_state;
Steven Valdez143e8b32016-07-11 13:19:03 -0400576 switch (state) {
Steven Valdez5440fe02016-07-18 12:40:30 -0400577 case state_process_hello_retry_request:
David Benjaminc3c88822016-11-14 10:32:04 +0900578 ret = do_process_hello_retry_request(hs);
Steven Valdez5440fe02016-07-18 12:40:30 -0400579 break;
580 case state_send_second_client_hello:
David Benjaminc3c88822016-11-14 10:32:04 +0900581 ret = do_send_second_client_hello(hs);
Steven Valdez5440fe02016-07-18 12:40:30 -0400582 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400583 case state_process_server_hello:
David Benjaminc3c88822016-11-14 10:32:04 +0900584 ret = do_process_server_hello(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400585 break;
586 case state_process_encrypted_extensions:
David Benjaminc3c88822016-11-14 10:32:04 +0900587 ret = do_process_encrypted_extensions(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400588 break;
589 case state_process_certificate_request:
David Benjaminc3c88822016-11-14 10:32:04 +0900590 ret = do_process_certificate_request(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400591 break;
592 case state_process_server_certificate:
David Benjaminc3c88822016-11-14 10:32:04 +0900593 ret = do_process_server_certificate(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400594 break;
595 case state_process_server_certificate_verify:
David Benjaminc3c88822016-11-14 10:32:04 +0900596 ret = do_process_server_certificate_verify(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400597 break;
598 case state_process_server_finished:
David Benjaminc3c88822016-11-14 10:32:04 +0900599 ret = do_process_server_finished(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400600 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400601 case state_send_client_certificate:
David Benjaminc3c88822016-11-14 10:32:04 +0900602 ret = do_send_client_certificate(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400603 break;
604 case state_send_client_certificate_verify:
David Benjaminc3c88822016-11-14 10:32:04 +0900605 ret = do_send_client_certificate_verify(hs, 1 /* first run */);
Nick Harper60a85cb2016-09-23 16:25:11 -0700606 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400607 case state_complete_client_certificate_verify:
David Benjaminc3c88822016-11-14 10:32:04 +0900608 ret = do_send_client_certificate_verify(hs, 0 /* complete */);
Nick Harper60a85cb2016-09-23 16:25:11 -0700609 break;
David Benjamin81b7bc32017-01-12 19:44:57 -0500610 case state_complete_second_flight:
611 ret = do_complete_second_flight(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400612 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400613 case state_done:
614 ret = ssl_hs_ok;
615 break;
616 }
617
618 if (ret != ssl_hs_ok) {
619 return ret;
620 }
621 }
622
623 return ssl_hs_ok;
624}
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400625
626int tls13_process_new_session_ticket(SSL *ssl) {
David Benjamin9c33ae82017-01-08 06:04:43 -0500627 int ret = 0;
Adam Langley46db7af2017-02-01 15:49:37 -0800628 SSL_SESSION *session = SSL_SESSION_dup(ssl->s3->established_session,
629 SSL_SESSION_INCLUDE_NONAUTH);
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400630 if (session == NULL) {
631 return 0;
632 }
633
David Benjamin17b30832017-01-28 14:00:32 -0500634 ssl_session_rebase_time(ssl, session);
David Benjamin123db572016-11-03 16:59:25 -0400635
David Benjamin17b30832017-01-28 14:00:32 -0500636 uint32_t server_timeout;
Steven Valdeza833c352016-11-01 13:39:36 -0400637 CBS cbs, ticket, extensions;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400638 CBS_init(&cbs, ssl->init_msg, ssl->init_num);
David Benjamin17b30832017-01-28 14:00:32 -0500639 if (!CBS_get_u32(&cbs, &server_timeout) ||
Steven Valdeza833c352016-11-01 13:39:36 -0400640 !CBS_get_u32(&cbs, &session->ticket_age_add) ||
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400641 !CBS_get_u16_length_prefixed(&cbs, &ticket) ||
642 !CBS_stow(&ticket, &session->tlsext_tick, &session->tlsext_ticklen) ||
Steven Valdez5b986082016-09-01 12:29:49 -0400643 !CBS_get_u16_length_prefixed(&cbs, &extensions) ||
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400644 CBS_len(&cbs) != 0) {
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400645 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
646 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
David Benjamin9c33ae82017-01-08 06:04:43 -0500647 goto err;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400648 }
649
David Benjamin17b30832017-01-28 14:00:32 -0500650 /* Cap the renewable lifetime by the server advertised value. This avoids
651 * wasting bandwidth on 0-RTT when we know the server will reject it.
652 *
653 * TODO(davidben): This dance where we're not sure if long or uint32_t is
654 * bigger is silly. session->timeout should not be a long to begin with.
655 * https://crbug.com/boringssl/155. */
656#if LONG_MAX < 0xffffffff
657 if (server_timeout > LONG_MAX) {
658 server_timeout = LONG_MAX;
659 }
660#endif
661 if (session->timeout > (long)server_timeout) {
662 session->timeout = (long)server_timeout;
663 }
664
Steven Valdez08b65f42016-12-07 15:29:45 -0500665 /* Parse out the extensions. */
666 int have_early_data_info = 0;
667 CBS early_data_info;
668 const SSL_EXTENSION_TYPE ext_types[] = {
669 {TLSEXT_TYPE_ticket_early_data_info, &have_early_data_info,
670 &early_data_info},
671 };
672
Adam Langleyc68e5b92017-02-08 13:33:15 -0800673 uint8_t alert = SSL_AD_DECODE_ERROR;
Steven Valdez08b65f42016-12-07 15:29:45 -0500674 if (!ssl_parse_extensions(&extensions, &alert, ext_types,
675 OPENSSL_ARRAY_SIZE(ext_types),
676 1 /* ignore unknown */)) {
677 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
David Benjamin9c33ae82017-01-08 06:04:43 -0500678 goto err;
Steven Valdez08b65f42016-12-07 15:29:45 -0500679 }
680
David Benjamin9b160662017-01-25 19:53:43 -0500681 if (have_early_data_info && ssl->ctx->enable_early_data) {
Steven Valdez08b65f42016-12-07 15:29:45 -0500682 if (!CBS_get_u32(&early_data_info, &session->ticket_max_early_data) ||
683 CBS_len(&early_data_info) != 0) {
684 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
David Benjamin9c33ae82017-01-08 06:04:43 -0500685 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
686 goto err;
Steven Valdez08b65f42016-12-07 15:29:45 -0500687 }
688 }
689
Steven Valdeza833c352016-11-01 13:39:36 -0400690 session->ticket_age_add_valid = 1;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400691 session->not_resumable = 0;
692
Steven Valdeza833c352016-11-01 13:39:36 -0400693 if (ssl->ctx->new_session_cb != NULL &&
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400694 ssl->ctx->new_session_cb(ssl, session)) {
695 /* |new_session_cb|'s return value signals that it took ownership. */
David Benjamin9c33ae82017-01-08 06:04:43 -0500696 session = NULL;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400697 }
698
David Benjamin9c33ae82017-01-08 06:04:43 -0500699 ret = 1;
700
701err:
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400702 SSL_SESSION_free(session);
David Benjamin9c33ae82017-01-08 06:04:43 -0500703 return ret;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400704}
David Benjamin4fe3c902016-08-16 02:17:03 -0400705
David Benjamin6e4fc332016-11-17 16:43:08 +0900706void ssl_clear_tls13_state(SSL_HANDSHAKE *hs) {
707 SSL_ECDH_CTX_cleanup(&hs->ecdh_ctx);
David Benjamin4fe3c902016-08-16 02:17:03 -0400708
David Benjamin6e4fc332016-11-17 16:43:08 +0900709 OPENSSL_free(hs->key_share_bytes);
710 hs->key_share_bytes = NULL;
711 hs->key_share_bytes_len = 0;
David Benjamin4fe3c902016-08-16 02:17:03 -0400712}