blob: ac70348d8e597b863e3111cd5b18169b82accb6b [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;
Adam Langley34b4c822017-02-02 10:57:17 -0800405 STACK_OF(CRYPTO_BUFFER) *ca_names =
406 ssl_parse_client_CA_list(ssl, &alert, &cbs);
407 if (ca_names == NULL) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400408 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
409 return ssl_hs_error;
410 }
411
412 /* Ignore extensions. */
413 CBS extensions;
414 if (!CBS_get_u16_length_prefixed(&cbs, &extensions) ||
415 CBS_len(&cbs) != 0) {
416 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
Adam Langley34b4c822017-02-02 10:57:17 -0800417 sk_CRYPTO_BUFFER_pop_free(ca_names, CRYPTO_BUFFER_free);
Steven Valdez143e8b32016-07-11 13:19:03 -0400418 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
419 return ssl_hs_error;
420 }
421
David Benjaminc3c88822016-11-14 10:32:04 +0900422 hs->cert_request = 1;
Adam Langley34b4c822017-02-02 10:57:17 -0800423 sk_CRYPTO_BUFFER_pop_free(hs->ca_names, CRYPTO_BUFFER_free);
424 hs->ca_names = ca_names;
425 ssl->ctx->x509_method->hs_flush_cached_ca_names(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400426
Steven Valdez908ac192017-01-12 13:17:07 -0500427 if (!ssl_hash_current_message(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400428 return ssl_hs_error;
429 }
430
David Benjamin3977f302016-12-11 13:30:41 -0500431 hs->tls13_state = state_process_server_certificate;
Steven Valdez143e8b32016-07-11 13:19:03 -0400432 return ssl_hs_read_message;
433}
434
David Benjaminc3c88822016-11-14 10:32:04 +0900435static enum ssl_hs_wait_t do_process_server_certificate(SSL_HANDSHAKE *hs) {
436 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500437 if (!ssl_check_message_type(ssl, SSL3_MT_CERTIFICATE) ||
Adam Langley0c294252016-12-12 11:46:09 -0800438 !tls13_process_certificate(hs, 0 /* certificate required */) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500439 !ssl_hash_current_message(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400440 return ssl_hs_error;
441 }
442
David Benjamin3977f302016-12-11 13:30:41 -0500443 hs->tls13_state = state_process_server_certificate_verify;
Steven Valdez143e8b32016-07-11 13:19:03 -0400444 return ssl_hs_read_message;
445}
446
447static enum ssl_hs_wait_t do_process_server_certificate_verify(
David Benjaminc3c88822016-11-14 10:32:04 +0900448 SSL_HANDSHAKE *hs) {
449 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500450 if (!ssl_check_message_type(ssl, SSL3_MT_CERTIFICATE_VERIFY) ||
Adam Langley0c294252016-12-12 11:46:09 -0800451 !tls13_process_certificate_verify(hs) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500452 !ssl_hash_current_message(hs)) {
David Benjamin6929f272016-11-16 19:10:08 +0900453 return ssl_hs_error;
Steven Valdez143e8b32016-07-11 13:19:03 -0400454 }
455
David Benjamin3977f302016-12-11 13:30:41 -0500456 hs->tls13_state = state_process_server_finished;
Steven Valdez143e8b32016-07-11 13:19:03 -0400457 return ssl_hs_read_message;
458}
459
David Benjaminc3c88822016-11-14 10:32:04 +0900460static enum ssl_hs_wait_t do_process_server_finished(SSL_HANDSHAKE *hs) {
461 SSL *const ssl = hs->ssl;
David Benjamin276b7e82017-01-21 14:13:39 -0500462 if (!ssl_check_message_type(ssl, SSL3_MT_FINISHED) ||
David Benjamin6e4fc332016-11-17 16:43:08 +0900463 !tls13_process_finished(hs) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500464 !ssl_hash_current_message(hs) ||
Steven Valdez143e8b32016-07-11 13:19:03 -0400465 /* Update the secret to the master secret and derive traffic keys. */
David Benjamin6e4fc332016-11-17 16:43:08 +0900466 !tls13_advance_key_schedule(hs, kZeroes, hs->hash_len) ||
467 !tls13_derive_application_secrets(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400468 return ssl_hs_error;
469 }
470
David Benjamin613fe3b2016-07-22 17:39:29 +0200471 ssl->method->received_flight(ssl);
David Benjamin5edfc8c2016-12-10 15:46:58 -0500472 hs->tls13_state = state_send_client_certificate;
Steven Valdez143e8b32016-07-11 13:19:03 -0400473 return ssl_hs_ok;
474}
475
David Benjamin5edfc8c2016-12-10 15:46:58 -0500476static enum ssl_hs_wait_t do_send_client_certificate(SSL_HANDSHAKE *hs) {
David Benjaminc3c88822016-11-14 10:32:04 +0900477 SSL *const ssl = hs->ssl;
Steven Valdez143e8b32016-07-11 13:19:03 -0400478 /* The peer didn't request a certificate. */
David Benjaminc3c88822016-11-14 10:32:04 +0900479 if (!hs->cert_request) {
David Benjamin81b7bc32017-01-12 19:44:57 -0500480 hs->tls13_state = state_complete_second_flight;
Steven Valdez143e8b32016-07-11 13:19:03 -0400481 return ssl_hs_ok;
482 }
483
484 /* Call cert_cb to update the certificate. */
485 if (ssl->cert->cert_cb != NULL) {
486 int rv = ssl->cert->cert_cb(ssl, ssl->cert->cert_cb_arg);
487 if (rv == 0) {
488 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR);
489 OPENSSL_PUT_ERROR(SSL, SSL_R_CERT_CB_ERROR);
490 return ssl_hs_error;
491 }
492 if (rv < 0) {
David Benjamin3977f302016-12-11 13:30:41 -0500493 hs->tls13_state = state_send_client_certificate;
Steven Valdez143e8b32016-07-11 13:19:03 -0400494 return ssl_hs_x509_lookup;
495 }
Steven Valdez143e8b32016-07-11 13:19:03 -0400496 }
497
David Benjamin650aa1c2016-12-20 18:55:16 -0500498 if (!ssl_auto_chain_if_needed(ssl) ||
David Benjamin0f24bed2017-01-12 19:46:50 -0500499 !tls13_add_certificate(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400500 return ssl_hs_error;
501 }
502
David Benjamin3977f302016-12-11 13:30:41 -0500503 hs->tls13_state = state_send_client_certificate_verify;
David Benjamin25ac2512017-01-12 19:31:28 -0500504 return ssl_hs_ok;
Steven Valdez143e8b32016-07-11 13:19:03 -0400505}
506
David Benjaminc3c88822016-11-14 10:32:04 +0900507static enum ssl_hs_wait_t do_send_client_certificate_verify(SSL_HANDSHAKE *hs,
Steven Valdez143e8b32016-07-11 13:19:03 -0400508 int is_first_run) {
David Benjaminc3c88822016-11-14 10:32:04 +0900509 SSL *const ssl = hs->ssl;
Steven Valdez143e8b32016-07-11 13:19:03 -0400510 /* Don't send CertificateVerify if there is no certificate. */
511 if (!ssl_has_certificate(ssl)) {
David Benjamin81b7bc32017-01-12 19:44:57 -0500512 hs->tls13_state = state_complete_second_flight;
Steven Valdez143e8b32016-07-11 13:19:03 -0400513 return ssl_hs_ok;
514 }
515
David Benjamin0f24bed2017-01-12 19:46:50 -0500516 switch (tls13_add_certificate_verify(hs, is_first_run)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400517 case ssl_private_key_success:
David Benjamin81b7bc32017-01-12 19:44:57 -0500518 hs->tls13_state = state_complete_second_flight;
David Benjamin25ac2512017-01-12 19:31:28 -0500519 return ssl_hs_ok;
Steven Valdez143e8b32016-07-11 13:19:03 -0400520
521 case ssl_private_key_retry:
David Benjamin3977f302016-12-11 13:30:41 -0500522 hs->tls13_state = state_complete_client_certificate_verify;
Steven Valdez143e8b32016-07-11 13:19:03 -0400523 return ssl_hs_private_key_operation;
524
525 case ssl_private_key_failure:
526 return ssl_hs_error;
527 }
528
529 assert(0);
530 return ssl_hs_error;
531}
532
David Benjamin81b7bc32017-01-12 19:44:57 -0500533static enum ssl_hs_wait_t do_complete_second_flight(SSL_HANDSHAKE *hs) {
David Benjaminc3c88822016-11-14 10:32:04 +0900534 SSL *const ssl = hs->ssl;
David Benjamin81b7bc32017-01-12 19:44:57 -0500535
536 /* Send a Channel ID assertion if necessary. */
537 if (ssl->s3->tlsext_channel_id_valid) {
538 if (!ssl_do_channel_id_callback(ssl)) {
539 hs->tls13_state = state_complete_second_flight;
540 return ssl_hs_error;
541 }
542
543 if (ssl->tlsext_channel_id_private == NULL) {
544 return ssl_hs_channel_id_lookup;
545 }
546
547 CBB cbb, body;
548 if (!ssl->method->init_message(ssl, &cbb, &body, SSL3_MT_CHANNEL_ID) ||
Steven Valdez908ac192017-01-12 13:17:07 -0500549 !tls1_write_channel_id(hs, &body) ||
David Benjamin81b7bc32017-01-12 19:44:57 -0500550 !ssl_add_message_cbb(ssl, &cbb)) {
551 CBB_cleanup(&cbb);
552 return ssl_hs_error;
553 }
Nick Harper60a85cb2016-09-23 16:25:11 -0700554 }
555
David Benjamin81b7bc32017-01-12 19:44:57 -0500556 /* Send a Finished message. */
David Benjamin0f24bed2017-01-12 19:46:50 -0500557 if (!tls13_add_finished(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400558 return ssl_hs_error;
559 }
560
David Benjamin81b7bc32017-01-12 19:44:57 -0500561 /* Derive the final keys and enable them. */
Steven Valdeza833c352016-11-01 13:39:36 -0400562 if (!tls13_set_traffic_key(ssl, evp_aead_open, hs->server_traffic_secret_0,
563 hs->hash_len) ||
564 !tls13_set_traffic_key(ssl, evp_aead_seal, hs->client_traffic_secret_0,
565 hs->hash_len) ||
David Benjamin6e4fc332016-11-17 16:43:08 +0900566 !tls13_derive_resumption_secret(hs)) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400567 return ssl_hs_error;
568 }
569
David Benjamin3977f302016-12-11 13:30:41 -0500570 hs->tls13_state = state_done;
Steven Valdez143e8b32016-07-11 13:19:03 -0400571 return ssl_hs_flush;
572}
573
David Benjaminc3c88822016-11-14 10:32:04 +0900574enum ssl_hs_wait_t tls13_client_handshake(SSL_HANDSHAKE *hs) {
David Benjamin3977f302016-12-11 13:30:41 -0500575 while (hs->tls13_state != state_done) {
Steven Valdez143e8b32016-07-11 13:19:03 -0400576 enum ssl_hs_wait_t ret = ssl_hs_error;
David Benjamin3977f302016-12-11 13:30:41 -0500577 enum client_hs_state_t state = hs->tls13_state;
Steven Valdez143e8b32016-07-11 13:19:03 -0400578 switch (state) {
Steven Valdez5440fe02016-07-18 12:40:30 -0400579 case state_process_hello_retry_request:
David Benjaminc3c88822016-11-14 10:32:04 +0900580 ret = do_process_hello_retry_request(hs);
Steven Valdez5440fe02016-07-18 12:40:30 -0400581 break;
582 case state_send_second_client_hello:
David Benjaminc3c88822016-11-14 10:32:04 +0900583 ret = do_send_second_client_hello(hs);
Steven Valdez5440fe02016-07-18 12:40:30 -0400584 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400585 case state_process_server_hello:
David Benjaminc3c88822016-11-14 10:32:04 +0900586 ret = do_process_server_hello(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400587 break;
588 case state_process_encrypted_extensions:
David Benjaminc3c88822016-11-14 10:32:04 +0900589 ret = do_process_encrypted_extensions(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400590 break;
591 case state_process_certificate_request:
David Benjaminc3c88822016-11-14 10:32:04 +0900592 ret = do_process_certificate_request(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400593 break;
594 case state_process_server_certificate:
David Benjaminc3c88822016-11-14 10:32:04 +0900595 ret = do_process_server_certificate(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400596 break;
597 case state_process_server_certificate_verify:
David Benjaminc3c88822016-11-14 10:32:04 +0900598 ret = do_process_server_certificate_verify(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400599 break;
600 case state_process_server_finished:
David Benjaminc3c88822016-11-14 10:32:04 +0900601 ret = do_process_server_finished(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400602 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400603 case state_send_client_certificate:
David Benjaminc3c88822016-11-14 10:32:04 +0900604 ret = do_send_client_certificate(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400605 break;
606 case state_send_client_certificate_verify:
David Benjaminc3c88822016-11-14 10:32:04 +0900607 ret = do_send_client_certificate_verify(hs, 1 /* first run */);
Nick Harper60a85cb2016-09-23 16:25:11 -0700608 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400609 case state_complete_client_certificate_verify:
David Benjaminc3c88822016-11-14 10:32:04 +0900610 ret = do_send_client_certificate_verify(hs, 0 /* complete */);
Nick Harper60a85cb2016-09-23 16:25:11 -0700611 break;
David Benjamin81b7bc32017-01-12 19:44:57 -0500612 case state_complete_second_flight:
613 ret = do_complete_second_flight(hs);
Steven Valdez143e8b32016-07-11 13:19:03 -0400614 break;
Steven Valdez143e8b32016-07-11 13:19:03 -0400615 case state_done:
616 ret = ssl_hs_ok;
617 break;
618 }
619
620 if (ret != ssl_hs_ok) {
621 return ret;
622 }
623 }
624
625 return ssl_hs_ok;
626}
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400627
628int tls13_process_new_session_ticket(SSL *ssl) {
David Benjamin9c33ae82017-01-08 06:04:43 -0500629 int ret = 0;
Adam Langley46db7af2017-02-01 15:49:37 -0800630 SSL_SESSION *session = SSL_SESSION_dup(ssl->s3->established_session,
631 SSL_SESSION_INCLUDE_NONAUTH);
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400632 if (session == NULL) {
633 return 0;
634 }
635
David Benjamin17b30832017-01-28 14:00:32 -0500636 ssl_session_rebase_time(ssl, session);
David Benjamin123db572016-11-03 16:59:25 -0400637
David Benjamin17b30832017-01-28 14:00:32 -0500638 uint32_t server_timeout;
Steven Valdeza833c352016-11-01 13:39:36 -0400639 CBS cbs, ticket, extensions;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400640 CBS_init(&cbs, ssl->init_msg, ssl->init_num);
David Benjamin17b30832017-01-28 14:00:32 -0500641 if (!CBS_get_u32(&cbs, &server_timeout) ||
Steven Valdeza833c352016-11-01 13:39:36 -0400642 !CBS_get_u32(&cbs, &session->ticket_age_add) ||
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400643 !CBS_get_u16_length_prefixed(&cbs, &ticket) ||
644 !CBS_stow(&ticket, &session->tlsext_tick, &session->tlsext_ticklen) ||
Steven Valdez5b986082016-09-01 12:29:49 -0400645 !CBS_get_u16_length_prefixed(&cbs, &extensions) ||
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400646 CBS_len(&cbs) != 0) {
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400647 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
648 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
David Benjamin9c33ae82017-01-08 06:04:43 -0500649 goto err;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400650 }
651
David Benjamin17b30832017-01-28 14:00:32 -0500652 /* Cap the renewable lifetime by the server advertised value. This avoids
653 * wasting bandwidth on 0-RTT when we know the server will reject it.
654 *
655 * TODO(davidben): This dance where we're not sure if long or uint32_t is
656 * bigger is silly. session->timeout should not be a long to begin with.
657 * https://crbug.com/boringssl/155. */
658#if LONG_MAX < 0xffffffff
659 if (server_timeout > LONG_MAX) {
660 server_timeout = LONG_MAX;
661 }
662#endif
663 if (session->timeout > (long)server_timeout) {
664 session->timeout = (long)server_timeout;
665 }
666
Steven Valdez08b65f42016-12-07 15:29:45 -0500667 /* Parse out the extensions. */
668 int have_early_data_info = 0;
669 CBS early_data_info;
670 const SSL_EXTENSION_TYPE ext_types[] = {
671 {TLSEXT_TYPE_ticket_early_data_info, &have_early_data_info,
672 &early_data_info},
673 };
674
Adam Langleyc68e5b92017-02-08 13:33:15 -0800675 uint8_t alert = SSL_AD_DECODE_ERROR;
Steven Valdez08b65f42016-12-07 15:29:45 -0500676 if (!ssl_parse_extensions(&extensions, &alert, ext_types,
677 OPENSSL_ARRAY_SIZE(ext_types),
678 1 /* ignore unknown */)) {
679 ssl3_send_alert(ssl, SSL3_AL_FATAL, alert);
David Benjamin9c33ae82017-01-08 06:04:43 -0500680 goto err;
Steven Valdez08b65f42016-12-07 15:29:45 -0500681 }
682
David Benjamin9b160662017-01-25 19:53:43 -0500683 if (have_early_data_info && ssl->ctx->enable_early_data) {
Steven Valdez08b65f42016-12-07 15:29:45 -0500684 if (!CBS_get_u32(&early_data_info, &session->ticket_max_early_data) ||
685 CBS_len(&early_data_info) != 0) {
686 ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR);
David Benjamin9c33ae82017-01-08 06:04:43 -0500687 OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR);
688 goto err;
Steven Valdez08b65f42016-12-07 15:29:45 -0500689 }
690 }
691
Steven Valdeza833c352016-11-01 13:39:36 -0400692 session->ticket_age_add_valid = 1;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400693 session->not_resumable = 0;
694
Steven Valdeza833c352016-11-01 13:39:36 -0400695 if (ssl->ctx->new_session_cb != NULL &&
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400696 ssl->ctx->new_session_cb(ssl, session)) {
697 /* |new_session_cb|'s return value signals that it took ownership. */
David Benjamin9c33ae82017-01-08 06:04:43 -0500698 session = NULL;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400699 }
700
David Benjamin9c33ae82017-01-08 06:04:43 -0500701 ret = 1;
702
703err:
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400704 SSL_SESSION_free(session);
David Benjamin9c33ae82017-01-08 06:04:43 -0500705 return ret;
Steven Valdez1e6f11a2016-07-27 11:10:52 -0400706}
David Benjamin4fe3c902016-08-16 02:17:03 -0400707
David Benjamin6e4fc332016-11-17 16:43:08 +0900708void ssl_clear_tls13_state(SSL_HANDSHAKE *hs) {
709 SSL_ECDH_CTX_cleanup(&hs->ecdh_ctx);
David Benjamin4fe3c902016-08-16 02:17:03 -0400710
David Benjamin6e4fc332016-11-17 16:43:08 +0900711 OPENSSL_free(hs->key_share_bytes);
712 hs->key_share_bytes = NULL;
713 hs->key_share_bytes_len = 0;
David Benjamin4fe3c902016-08-16 02:17:03 -0400714}