Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 1 | /* 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> |
| 18 | #include <string.h> |
| 19 | |
| 20 | #include <openssl/bytestring.h> |
| 21 | #include <openssl/digest.h> |
| 22 | #include <openssl/err.h> |
| 23 | #include <openssl/mem.h> |
| 24 | #include <openssl/stack.h> |
| 25 | #include <openssl/x509.h> |
| 26 | |
David Benjamin | ffb1107 | 2016-11-13 10:32:10 +0900 | [diff] [blame^] | 27 | #include "../crypto/internal.h" |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 28 | #include "internal.h" |
| 29 | |
| 30 | |
| 31 | enum client_hs_state_t { |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 32 | state_process_hello_retry_request = 0, |
| 33 | state_send_second_client_hello, |
| 34 | state_flush_second_client_hello, |
| 35 | state_process_server_hello, |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 36 | 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, |
| 41 | state_certificate_callback, |
| 42 | state_send_client_certificate, |
| 43 | state_send_client_certificate_verify, |
| 44 | state_complete_client_certificate_verify, |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 45 | state_send_channel_id, |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 46 | state_send_client_finished, |
| 47 | state_flush, |
| 48 | state_done, |
| 49 | }; |
| 50 | |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 51 | static enum ssl_hs_wait_t do_process_hello_retry_request(SSL *ssl, |
| 52 | SSL_HANDSHAKE *hs) { |
| 53 | if (ssl->s3->tmp.message_type != SSL3_MT_HELLO_RETRY_REQUEST) { |
| 54 | hs->state = state_process_server_hello; |
| 55 | return ssl_hs_ok; |
| 56 | } |
| 57 | |
| 58 | CBS cbs, extensions; |
David Benjamin | 3baa6e1 | 2016-10-07 21:10:38 -0400 | [diff] [blame] | 59 | uint16_t server_wire_version; |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 60 | CBS_init(&cbs, ssl->init_msg, ssl->init_num); |
| 61 | if (!CBS_get_u16(&cbs, &server_wire_version) || |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 62 | !CBS_get_u16_length_prefixed(&cbs, &extensions) || |
David Benjamin | 3baa6e1 | 2016-10-07 21:10:38 -0400 | [diff] [blame] | 63 | /* HelloRetryRequest may not be empty. */ |
| 64 | CBS_len(&extensions) == 0 || |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 65 | CBS_len(&cbs) != 0) { |
David Benjamin | 3baa6e1 | 2016-10-07 21:10:38 -0400 | [diff] [blame] | 66 | OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR); |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 67 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR); |
| 68 | return ssl_hs_error; |
| 69 | } |
| 70 | |
David Benjamin | ffb1107 | 2016-11-13 10:32:10 +0900 | [diff] [blame^] | 71 | int have_cookie, have_key_share; |
| 72 | CBS cookie, key_share; |
| 73 | const SSL_EXTENSION_TYPE ext_types[] = { |
| 74 | {TLSEXT_TYPE_key_share, &have_key_share, &key_share}, |
| 75 | {TLSEXT_TYPE_cookie, &have_cookie, &cookie}, |
| 76 | }; |
| 77 | |
| 78 | uint8_t alert; |
| 79 | if (!ssl_parse_extensions(&extensions, &alert, ext_types, |
| 80 | OPENSSL_ARRAY_SIZE(ext_types))) { |
| 81 | 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 Benjamin | 3baa6e1 | 2016-10-07 21:10:38 -0400 | [diff] [blame] | 90 | 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 Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 94 | |
David Benjamin | ffb1107 | 2016-11-13 10:32:10 +0900 | [diff] [blame^] | 95 | if (!CBS_stow(&cookie_value, &hs->cookie, &hs->cookie_len)) { |
| 96 | return ssl_hs_error; |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 97 | } |
| 98 | } |
| 99 | |
David Benjamin | ffb1107 | 2016-11-13 10:32:10 +0900 | [diff] [blame^] | 100 | 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 | |
David Benjamin | 3baa6e1 | 2016-10-07 21:10:38 -0400 | [diff] [blame] | 138 | hs->received_hello_retry_request = 1; |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 139 | hs->state = state_send_second_client_hello; |
| 140 | return ssl_hs_ok; |
| 141 | } |
| 142 | |
| 143 | static enum ssl_hs_wait_t do_send_second_client_hello(SSL *ssl, |
| 144 | SSL_HANDSHAKE *hs) { |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 145 | if (!ssl_write_client_hello(ssl)) { |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 146 | return ssl_hs_error; |
| 147 | } |
| 148 | |
| 149 | hs->state = state_flush_second_client_hello; |
| 150 | return ssl_hs_write_message; |
| 151 | } |
| 152 | |
| 153 | static enum ssl_hs_wait_t do_flush_second_client_hello(SSL *ssl, |
| 154 | SSL_HANDSHAKE *hs) { |
| 155 | hs->state = state_process_server_hello; |
| 156 | return ssl_hs_flush_and_read_message; |
| 157 | } |
| 158 | |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 159 | static enum ssl_hs_wait_t do_process_server_hello(SSL *ssl, SSL_HANDSHAKE *hs) { |
| 160 | if (!tls13_check_message_type(ssl, SSL3_MT_SERVER_HELLO)) { |
| 161 | return ssl_hs_error; |
| 162 | } |
| 163 | |
| 164 | CBS cbs, server_random, extensions; |
| 165 | uint16_t server_wire_version; |
| 166 | uint16_t cipher_suite; |
| 167 | CBS_init(&cbs, ssl->init_msg, ssl->init_num); |
| 168 | if (!CBS_get_u16(&cbs, &server_wire_version) || |
| 169 | !CBS_get_bytes(&cbs, &server_random, SSL3_RANDOM_SIZE) || |
| 170 | !CBS_get_u16(&cbs, &cipher_suite) || |
| 171 | !CBS_get_u16_length_prefixed(&cbs, &extensions) || |
| 172 | CBS_len(&cbs) != 0) { |
| 173 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR); |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 174 | OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR); |
| 175 | return ssl_hs_error; |
| 176 | } |
| 177 | |
| 178 | if (server_wire_version != ssl->version) { |
| 179 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR); |
| 180 | OPENSSL_PUT_ERROR(SSL, SSL_R_WRONG_VERSION_NUMBER); |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 181 | return ssl_hs_error; |
| 182 | } |
| 183 | |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 184 | assert(ssl->s3->have_version); |
| 185 | memcpy(ssl->s3->server_random, CBS_data(&server_random), SSL3_RANDOM_SIZE); |
| 186 | |
| 187 | const SSL_CIPHER *cipher = SSL_get_cipher_by_value(cipher_suite); |
| 188 | if (cipher == NULL) { |
| 189 | OPENSSL_PUT_ERROR(SSL, SSL_R_UNKNOWN_CIPHER_RETURNED); |
| 190 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER); |
| 191 | return ssl_hs_error; |
| 192 | } |
| 193 | |
David Benjamin | abbbee1 | 2016-10-31 19:20:42 -0400 | [diff] [blame] | 194 | /* Check if the cipher is a TLS 1.3 cipher. */ |
| 195 | if (SSL_CIPHER_get_min_version(cipher) > ssl3_protocol_version(ssl) || |
| 196 | SSL_CIPHER_get_max_version(cipher) < ssl3_protocol_version(ssl)) { |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 197 | OPENSSL_PUT_ERROR(SSL, SSL_R_WRONG_CIPHER_RETURNED); |
| 198 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER); |
| 199 | return ssl_hs_error; |
| 200 | } |
| 201 | |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 202 | /* Parse out the extensions. */ |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 203 | int have_key_share = 0, have_pre_shared_key = 0; |
| 204 | CBS key_share, pre_shared_key; |
David Benjamin | ffb1107 | 2016-11-13 10:32:10 +0900 | [diff] [blame^] | 205 | const SSL_EXTENSION_TYPE ext_types[] = { |
| 206 | {TLSEXT_TYPE_key_share, &have_key_share, &key_share}, |
| 207 | {TLSEXT_TYPE_pre_shared_key, &have_pre_shared_key, &pre_shared_key}, |
| 208 | }; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 209 | |
David Benjamin | ffb1107 | 2016-11-13 10:32:10 +0900 | [diff] [blame^] | 210 | uint8_t alert; |
| 211 | if (!ssl_parse_extensions(&extensions, &alert, ext_types, |
| 212 | OPENSSL_ARRAY_SIZE(ext_types))) { |
| 213 | ssl3_send_alert(ssl, SSL3_AL_FATAL, alert); |
| 214 | return ssl_hs_error; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 215 | } |
| 216 | |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 217 | /* We only support PSK_DHE_KE. */ |
| 218 | if (!have_key_share) { |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 219 | OPENSSL_PUT_ERROR(SSL, SSL_R_UNEXPECTED_EXTENSION); |
| 220 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER); |
| 221 | return ssl_hs_error; |
| 222 | } |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 223 | |
David Benjamin | ffb1107 | 2016-11-13 10:32:10 +0900 | [diff] [blame^] | 224 | alert = SSL_AD_DECODE_ERROR; |
Steven Valdez | 4aa154e | 2016-07-29 14:32:55 -0400 | [diff] [blame] | 225 | if (have_pre_shared_key) { |
| 226 | if (ssl->session == NULL) { |
| 227 | OPENSSL_PUT_ERROR(SSL, SSL_R_UNEXPECTED_EXTENSION); |
| 228 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_UNSUPPORTED_EXTENSION); |
| 229 | return ssl_hs_error; |
| 230 | } |
| 231 | |
| 232 | if (!ssl_ext_pre_shared_key_parse_serverhello(ssl, &alert, |
| 233 | &pre_shared_key)) { |
| 234 | ssl3_send_alert(ssl, SSL3_AL_FATAL, alert); |
| 235 | return ssl_hs_error; |
| 236 | } |
| 237 | |
| 238 | if (ssl->session->ssl_version != ssl->version) { |
| 239 | OPENSSL_PUT_ERROR(SSL, SSL_R_OLD_SESSION_VERSION_NOT_RETURNED); |
| 240 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER); |
| 241 | return ssl_hs_error; |
| 242 | } |
| 243 | |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 244 | if (ssl->session->cipher != cipher) { |
| 245 | OPENSSL_PUT_ERROR(SSL, SSL_R_OLD_SESSION_CIPHER_NOT_RETURNED); |
| 246 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER); |
| 247 | return ssl_hs_error; |
| 248 | } |
| 249 | |
Steven Valdez | 4aa154e | 2016-07-29 14:32:55 -0400 | [diff] [blame] | 250 | if (!ssl_session_is_context_valid(ssl, ssl->session)) { |
| 251 | /* This is actually a client application bug. */ |
| 252 | OPENSSL_PUT_ERROR(SSL, |
| 253 | SSL_R_ATTEMPT_TO_REUSE_SESSION_IN_DIFFERENT_CONTEXT); |
| 254 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_ILLEGAL_PARAMETER); |
| 255 | return ssl_hs_error; |
| 256 | } |
| 257 | |
| 258 | ssl->s3->session_reused = 1; |
| 259 | /* Only authentication information carries over in TLS 1.3. */ |
| 260 | ssl->s3->new_session = |
| 261 | SSL_SESSION_dup(ssl->session, SSL_SESSION_DUP_AUTH_ONLY); |
| 262 | if (ssl->s3->new_session == NULL) { |
| 263 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR); |
| 264 | return ssl_hs_error; |
| 265 | } |
David Benjamin | 4d0be24 | 2016-09-01 01:10:07 -0400 | [diff] [blame] | 266 | ssl_set_session(ssl, NULL); |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 267 | } else if (!ssl_get_new_session(ssl, 0)) { |
| 268 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR); |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 269 | return ssl_hs_error; |
| 270 | } |
| 271 | |
Steven Valdez | 87eab49 | 2016-06-27 16:34:59 -0400 | [diff] [blame] | 272 | ssl->s3->new_session->cipher = cipher; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 273 | ssl->s3->tmp.new_cipher = cipher; |
| 274 | |
| 275 | /* The PRF hash is now known. Set up the key schedule. */ |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 276 | size_t hash_len = |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 277 | EVP_MD_size(ssl_get_handshake_digest(ssl_get_algorithm_prf(ssl))); |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 278 | |
| 279 | /* Derive resumption material. */ |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 280 | uint8_t psk_secret[EVP_MAX_MD_SIZE] = {0}; |
Steven Valdez | 4aa154e | 2016-07-29 14:32:55 -0400 | [diff] [blame] | 281 | if (ssl->s3->session_reused) { |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 282 | if (hash_len != (size_t) ssl->s3->new_session->master_key_length) { |
Steven Valdez | 4aa154e | 2016-07-29 14:32:55 -0400 | [diff] [blame] | 283 | return ssl_hs_error; |
| 284 | } |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 285 | memcpy(psk_secret, ssl->s3->new_session->master_key, hash_len); |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 286 | } |
| 287 | |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 288 | /* Set up the key schedule, hash in the ClientHello, and incorporate the PSK |
| 289 | * into the running secret. */ |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 290 | if (!tls13_init_key_schedule(ssl) || |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 291 | !tls13_advance_key_schedule(ssl, psk_secret, hash_len)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 292 | return ssl_hs_error; |
| 293 | } |
| 294 | |
| 295 | /* Resolve ECDHE and incorporate it into the secret. */ |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 296 | uint8_t *dhe_secret; |
| 297 | size_t dhe_secret_len; |
| 298 | if (!ssl_ext_key_share_parse_serverhello(ssl, &dhe_secret, &dhe_secret_len, |
| 299 | &alert, &key_share)) { |
| 300 | ssl3_send_alert(ssl, SSL3_AL_FATAL, alert); |
| 301 | return ssl_hs_error; |
| 302 | } |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 303 | |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 304 | if (!tls13_advance_key_schedule(ssl, dhe_secret, dhe_secret_len)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 305 | OPENSSL_free(dhe_secret); |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 306 | return ssl_hs_error; |
| 307 | } |
| 308 | OPENSSL_free(dhe_secret); |
| 309 | |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 310 | /* If there was no HelloRetryRequest, the version negotiation logic has |
| 311 | * already hashed the message. */ |
David Benjamin | 3baa6e1 | 2016-10-07 21:10:38 -0400 | [diff] [blame] | 312 | if (hs->received_hello_retry_request && |
David Benjamin | ced9479 | 2016-11-14 17:12:11 +0900 | [diff] [blame] | 313 | !ssl_hash_current_message(ssl)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 314 | return ssl_hs_error; |
| 315 | } |
| 316 | |
| 317 | if (!tls13_set_handshake_traffic(ssl)) { |
| 318 | return ssl_hs_error; |
| 319 | } |
| 320 | |
| 321 | hs->state = state_process_encrypted_extensions; |
| 322 | return ssl_hs_read_message; |
| 323 | } |
| 324 | |
| 325 | static enum ssl_hs_wait_t do_process_encrypted_extensions(SSL *ssl, |
| 326 | SSL_HANDSHAKE *hs) { |
| 327 | if (!tls13_check_message_type(ssl, SSL3_MT_ENCRYPTED_EXTENSIONS)) { |
| 328 | return ssl_hs_error; |
| 329 | } |
| 330 | |
| 331 | CBS cbs; |
| 332 | CBS_init(&cbs, ssl->init_msg, ssl->init_num); |
| 333 | if (!ssl_parse_serverhello_tlsext(ssl, &cbs)) { |
| 334 | OPENSSL_PUT_ERROR(SSL, SSL_R_PARSE_TLSEXT); |
| 335 | return ssl_hs_error; |
| 336 | } |
| 337 | if (CBS_len(&cbs) != 0) { |
| 338 | OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR); |
| 339 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR); |
| 340 | return ssl_hs_error; |
| 341 | } |
| 342 | |
David Benjamin | ced9479 | 2016-11-14 17:12:11 +0900 | [diff] [blame] | 343 | if (!ssl_hash_current_message(ssl)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 344 | return ssl_hs_error; |
| 345 | } |
| 346 | |
| 347 | hs->state = state_process_certificate_request; |
| 348 | return ssl_hs_read_message; |
| 349 | } |
| 350 | |
| 351 | static enum ssl_hs_wait_t do_process_certificate_request(SSL *ssl, |
| 352 | SSL_HANDSHAKE *hs) { |
Steven Valdez | 803c77a | 2016-09-06 14:13:43 -0400 | [diff] [blame] | 353 | /* CertificateRequest may only be sent in non-resumption handshakes. */ |
| 354 | if (ssl->s3->session_reused) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 355 | hs->state = state_process_server_finished; |
| 356 | return ssl_hs_ok; |
| 357 | } |
| 358 | |
| 359 | /* CertificateRequest is optional. */ |
| 360 | if (ssl->s3->tmp.message_type != SSL3_MT_CERTIFICATE_REQUEST) { |
| 361 | hs->state = state_process_server_certificate; |
| 362 | return ssl_hs_ok; |
| 363 | } |
| 364 | |
| 365 | CBS cbs, context, supported_signature_algorithms; |
| 366 | CBS_init(&cbs, ssl->init_msg, ssl->init_num); |
| 367 | if (!CBS_get_u8_length_prefixed(&cbs, &context) || |
David Benjamin | 8a8349b | 2016-08-18 02:32:23 -0400 | [diff] [blame] | 368 | /* The request context is always empty during the handshake. */ |
| 369 | CBS_len(&context) != 0 || |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 370 | !CBS_get_u16_length_prefixed(&cbs, &supported_signature_algorithms) || |
David Benjamin | 4890165 | 2016-08-01 12:12:47 -0400 | [diff] [blame] | 371 | CBS_len(&supported_signature_algorithms) == 0 || |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 372 | !tls1_parse_peer_sigalgs(ssl, &supported_signature_algorithms)) { |
| 373 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR); |
| 374 | OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR); |
| 375 | return ssl_hs_error; |
| 376 | } |
| 377 | |
| 378 | uint8_t alert; |
| 379 | STACK_OF(X509_NAME) *ca_sk = ssl_parse_client_CA_list(ssl, &alert, &cbs); |
| 380 | if (ca_sk == NULL) { |
| 381 | ssl3_send_alert(ssl, SSL3_AL_FATAL, alert); |
| 382 | return ssl_hs_error; |
| 383 | } |
| 384 | |
| 385 | /* Ignore extensions. */ |
| 386 | CBS extensions; |
| 387 | if (!CBS_get_u16_length_prefixed(&cbs, &extensions) || |
| 388 | CBS_len(&cbs) != 0) { |
| 389 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR); |
David Benjamin | 639846e | 2016-09-09 11:41:18 -0400 | [diff] [blame] | 390 | sk_X509_NAME_pop_free(ca_sk, X509_NAME_free); |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 391 | OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR); |
| 392 | return ssl_hs_error; |
| 393 | } |
| 394 | |
David Benjamin | a048678 | 2016-10-06 19:11:32 -0400 | [diff] [blame] | 395 | ssl->s3->hs->cert_request = 1; |
| 396 | sk_X509_NAME_pop_free(ssl->s3->hs->ca_names, X509_NAME_free); |
| 397 | ssl->s3->hs->ca_names = ca_sk; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 398 | |
David Benjamin | ced9479 | 2016-11-14 17:12:11 +0900 | [diff] [blame] | 399 | if (!ssl_hash_current_message(ssl)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 400 | return ssl_hs_error; |
| 401 | } |
| 402 | |
| 403 | hs->state = state_process_server_certificate; |
| 404 | return ssl_hs_read_message; |
| 405 | } |
| 406 | |
| 407 | static enum ssl_hs_wait_t do_process_server_certificate(SSL *ssl, |
| 408 | SSL_HANDSHAKE *hs) { |
| 409 | if (!tls13_check_message_type(ssl, SSL3_MT_CERTIFICATE) || |
David Benjamin | 4087df9 | 2016-08-01 20:16:31 -0400 | [diff] [blame] | 410 | !tls13_process_certificate(ssl, 0 /* certificate required */) || |
David Benjamin | ced9479 | 2016-11-14 17:12:11 +0900 | [diff] [blame] | 411 | !ssl_hash_current_message(ssl)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 412 | return ssl_hs_error; |
| 413 | } |
| 414 | |
| 415 | hs->state = state_process_server_certificate_verify; |
| 416 | return ssl_hs_read_message; |
| 417 | } |
| 418 | |
| 419 | static enum ssl_hs_wait_t do_process_server_certificate_verify( |
| 420 | SSL *ssl, SSL_HANDSHAKE *hs) { |
| 421 | if (!tls13_check_message_type(ssl, SSL3_MT_CERTIFICATE_VERIFY) || |
| 422 | !tls13_process_certificate_verify(ssl) || |
David Benjamin | ced9479 | 2016-11-14 17:12:11 +0900 | [diff] [blame] | 423 | !ssl_hash_current_message(ssl)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 424 | return 0; |
| 425 | } |
| 426 | |
| 427 | hs->state = state_process_server_finished; |
| 428 | return ssl_hs_read_message; |
| 429 | } |
| 430 | |
| 431 | static enum ssl_hs_wait_t do_process_server_finished(SSL *ssl, |
| 432 | SSL_HANDSHAKE *hs) { |
| 433 | static const uint8_t kZeroes[EVP_MAX_MD_SIZE] = {0}; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 434 | if (!tls13_check_message_type(ssl, SSL3_MT_FINISHED) || |
| 435 | !tls13_process_finished(ssl) || |
David Benjamin | ced9479 | 2016-11-14 17:12:11 +0900 | [diff] [blame] | 436 | !ssl_hash_current_message(ssl) || |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 437 | /* Update the secret to the master secret and derive traffic keys. */ |
| 438 | !tls13_advance_key_schedule(ssl, kZeroes, hs->hash_len) || |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 439 | !tls13_derive_application_secrets(ssl)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 440 | return ssl_hs_error; |
| 441 | } |
| 442 | |
David Benjamin | 613fe3b | 2016-07-22 17:39:29 +0200 | [diff] [blame] | 443 | ssl->method->received_flight(ssl); |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 444 | hs->state = state_certificate_callback; |
| 445 | return ssl_hs_ok; |
| 446 | } |
| 447 | |
| 448 | static enum ssl_hs_wait_t do_certificate_callback(SSL *ssl, SSL_HANDSHAKE *hs) { |
| 449 | /* The peer didn't request a certificate. */ |
David Benjamin | a048678 | 2016-10-06 19:11:32 -0400 | [diff] [blame] | 450 | if (!ssl->s3->hs->cert_request) { |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 451 | hs->state = state_send_channel_id; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 452 | return ssl_hs_ok; |
| 453 | } |
| 454 | |
| 455 | /* Call cert_cb to update the certificate. */ |
| 456 | if (ssl->cert->cert_cb != NULL) { |
| 457 | int rv = ssl->cert->cert_cb(ssl, ssl->cert->cert_cb_arg); |
| 458 | if (rv == 0) { |
| 459 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_INTERNAL_ERROR); |
| 460 | OPENSSL_PUT_ERROR(SSL, SSL_R_CERT_CB_ERROR); |
| 461 | return ssl_hs_error; |
| 462 | } |
| 463 | if (rv < 0) { |
| 464 | hs->state = state_certificate_callback; |
| 465 | return ssl_hs_x509_lookup; |
| 466 | } |
| 467 | } |
| 468 | |
| 469 | hs->state = state_send_client_certificate; |
| 470 | return ssl_hs_ok; |
| 471 | } |
| 472 | |
| 473 | static enum ssl_hs_wait_t do_send_client_certificate(SSL *ssl, |
| 474 | SSL_HANDSHAKE *hs) { |
David Benjamin | 13f1ebe | 2016-07-20 10:11:04 +0200 | [diff] [blame] | 475 | /* Call client_cert_cb to update the certificate. */ |
| 476 | int should_retry; |
| 477 | if (!ssl_do_client_cert_cb(ssl, &should_retry)) { |
| 478 | if (should_retry) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 479 | hs->state = state_send_client_certificate; |
| 480 | return ssl_hs_x509_lookup; |
| 481 | } |
David Benjamin | 13f1ebe | 2016-07-20 10:11:04 +0200 | [diff] [blame] | 482 | return ssl_hs_error; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 483 | } |
| 484 | |
| 485 | if (!tls13_prepare_certificate(ssl)) { |
| 486 | return ssl_hs_error; |
| 487 | } |
| 488 | |
| 489 | hs->state = state_send_client_certificate_verify; |
| 490 | return ssl_hs_write_message; |
| 491 | } |
| 492 | |
| 493 | static enum ssl_hs_wait_t do_send_client_certificate_verify(SSL *ssl, |
| 494 | SSL_HANDSHAKE *hs, |
| 495 | int is_first_run) { |
| 496 | /* Don't send CertificateVerify if there is no certificate. */ |
| 497 | if (!ssl_has_certificate(ssl)) { |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 498 | hs->state = state_send_channel_id; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 499 | return ssl_hs_ok; |
| 500 | } |
| 501 | |
| 502 | switch (tls13_prepare_certificate_verify(ssl, is_first_run)) { |
| 503 | case ssl_private_key_success: |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 504 | hs->state = state_send_channel_id; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 505 | return ssl_hs_write_message; |
| 506 | |
| 507 | case ssl_private_key_retry: |
| 508 | hs->state = state_complete_client_certificate_verify; |
| 509 | return ssl_hs_private_key_operation; |
| 510 | |
| 511 | case ssl_private_key_failure: |
| 512 | return ssl_hs_error; |
| 513 | } |
| 514 | |
| 515 | assert(0); |
| 516 | return ssl_hs_error; |
| 517 | } |
| 518 | |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 519 | static enum ssl_hs_wait_t do_send_channel_id(SSL *ssl, SSL_HANDSHAKE *hs) { |
| 520 | if (!ssl->s3->tlsext_channel_id_valid) { |
| 521 | hs->state = state_send_client_finished; |
| 522 | return ssl_hs_ok; |
| 523 | } |
| 524 | |
| 525 | if (!ssl_do_channel_id_callback(ssl)) { |
| 526 | return ssl_hs_error; |
| 527 | } |
| 528 | |
| 529 | if (ssl->tlsext_channel_id_private == NULL) { |
| 530 | return ssl_hs_channel_id_lookup; |
| 531 | } |
| 532 | |
| 533 | CBB cbb, body; |
| 534 | if (!ssl->method->init_message(ssl, &cbb, &body, SSL3_MT_CHANNEL_ID) || |
| 535 | !tls1_write_channel_id(ssl, &body) || |
Steven Valdez | 5eead16 | 2016-11-11 22:23:25 -0500 | [diff] [blame] | 536 | !ssl_complete_message(ssl, &cbb)) { |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 537 | CBB_cleanup(&cbb); |
| 538 | return ssl_hs_error; |
| 539 | } |
| 540 | |
| 541 | hs->state = state_send_client_finished; |
| 542 | return ssl_hs_write_message; |
| 543 | } |
| 544 | |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 545 | static enum ssl_hs_wait_t do_send_client_finished(SSL *ssl, SSL_HANDSHAKE *hs) { |
| 546 | if (!tls13_prepare_finished(ssl)) { |
| 547 | return ssl_hs_error; |
| 548 | } |
| 549 | |
| 550 | hs->state = state_flush; |
| 551 | return ssl_hs_write_message; |
| 552 | } |
| 553 | |
| 554 | static enum ssl_hs_wait_t do_flush(SSL *ssl, SSL_HANDSHAKE *hs) { |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 555 | if (!tls13_set_traffic_key(ssl, evp_aead_open, hs->server_traffic_secret_0, |
| 556 | hs->hash_len) || |
| 557 | !tls13_set_traffic_key(ssl, evp_aead_seal, hs->client_traffic_secret_0, |
| 558 | hs->hash_len) || |
| 559 | !tls13_derive_resumption_secret(ssl)) { |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 560 | return ssl_hs_error; |
| 561 | } |
| 562 | |
| 563 | hs->state = state_done; |
| 564 | return ssl_hs_flush; |
| 565 | } |
| 566 | |
| 567 | enum ssl_hs_wait_t tls13_client_handshake(SSL *ssl) { |
| 568 | SSL_HANDSHAKE *hs = ssl->s3->hs; |
| 569 | |
| 570 | while (hs->state != state_done) { |
| 571 | enum ssl_hs_wait_t ret = ssl_hs_error; |
| 572 | enum client_hs_state_t state = hs->state; |
| 573 | switch (state) { |
Steven Valdez | 5440fe0 | 2016-07-18 12:40:30 -0400 | [diff] [blame] | 574 | case state_process_hello_retry_request: |
| 575 | ret = do_process_hello_retry_request(ssl, hs); |
| 576 | break; |
| 577 | case state_send_second_client_hello: |
| 578 | ret = do_send_second_client_hello(ssl, hs); |
| 579 | break; |
| 580 | case state_flush_second_client_hello: |
| 581 | ret = do_flush_second_client_hello(ssl, hs); |
| 582 | break; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 583 | case state_process_server_hello: |
| 584 | ret = do_process_server_hello(ssl, hs); |
| 585 | break; |
| 586 | case state_process_encrypted_extensions: |
| 587 | ret = do_process_encrypted_extensions(ssl, hs); |
| 588 | break; |
| 589 | case state_process_certificate_request: |
| 590 | ret = do_process_certificate_request(ssl, hs); |
| 591 | break; |
| 592 | case state_process_server_certificate: |
| 593 | ret = do_process_server_certificate(ssl, hs); |
| 594 | break; |
| 595 | case state_process_server_certificate_verify: |
| 596 | ret = do_process_server_certificate_verify(ssl, hs); |
| 597 | break; |
| 598 | case state_process_server_finished: |
| 599 | ret = do_process_server_finished(ssl, hs); |
| 600 | break; |
| 601 | case state_certificate_callback: |
| 602 | ret = do_certificate_callback(ssl, hs); |
| 603 | break; |
| 604 | case state_send_client_certificate: |
| 605 | ret = do_send_client_certificate(ssl, hs); |
| 606 | break; |
| 607 | case state_send_client_certificate_verify: |
| 608 | ret = do_send_client_certificate_verify(ssl, hs, 1 /* first run */); |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 609 | break; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 610 | case state_complete_client_certificate_verify: |
| 611 | ret = do_send_client_certificate_verify(ssl, hs, 0 /* complete */); |
Nick Harper | 60a85cb | 2016-09-23 16:25:11 -0700 | [diff] [blame] | 612 | break; |
| 613 | case state_send_channel_id: |
| 614 | ret = do_send_channel_id(ssl, hs); |
| 615 | break; |
Steven Valdez | 143e8b3 | 2016-07-11 13:19:03 -0400 | [diff] [blame] | 616 | case state_send_client_finished: |
| 617 | ret = do_send_client_finished(ssl, hs); |
| 618 | break; |
| 619 | case state_flush: |
| 620 | ret = do_flush(ssl, hs); |
| 621 | break; |
| 622 | case state_done: |
| 623 | ret = ssl_hs_ok; |
| 624 | break; |
| 625 | } |
| 626 | |
| 627 | if (ret != ssl_hs_ok) { |
| 628 | return ret; |
| 629 | } |
| 630 | } |
| 631 | |
| 632 | return ssl_hs_ok; |
| 633 | } |
Steven Valdez | 1e6f11a | 2016-07-27 11:10:52 -0400 | [diff] [blame] | 634 | |
| 635 | int tls13_process_new_session_ticket(SSL *ssl) { |
Steven Valdez | 4aa154e | 2016-07-29 14:32:55 -0400 | [diff] [blame] | 636 | SSL_SESSION *session = |
| 637 | SSL_SESSION_dup(ssl->s3->established_session, |
| 638 | SSL_SESSION_INCLUDE_NONAUTH); |
Steven Valdez | 1e6f11a | 2016-07-27 11:10:52 -0400 | [diff] [blame] | 639 | if (session == NULL) { |
| 640 | return 0; |
| 641 | } |
| 642 | |
David Benjamin | 123db57 | 2016-11-03 16:59:25 -0400 | [diff] [blame] | 643 | ssl_session_refresh_time(ssl, session); |
| 644 | |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 645 | CBS cbs, ticket, extensions; |
Steven Valdez | 1e6f11a | 2016-07-27 11:10:52 -0400 | [diff] [blame] | 646 | CBS_init(&cbs, ssl->init_msg, ssl->init_num); |
Martin Kreichgauer | baafa4a | 2016-08-09 10:18:40 -0700 | [diff] [blame] | 647 | if (!CBS_get_u32(&cbs, &session->tlsext_tick_lifetime_hint) || |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 648 | !CBS_get_u32(&cbs, &session->ticket_age_add) || |
Steven Valdez | 1e6f11a | 2016-07-27 11:10:52 -0400 | [diff] [blame] | 649 | !CBS_get_u16_length_prefixed(&cbs, &ticket) || |
| 650 | !CBS_stow(&ticket, &session->tlsext_tick, &session->tlsext_ticklen) || |
Steven Valdez | 5b98608 | 2016-09-01 12:29:49 -0400 | [diff] [blame] | 651 | !CBS_get_u16_length_prefixed(&cbs, &extensions) || |
Steven Valdez | 1e6f11a | 2016-07-27 11:10:52 -0400 | [diff] [blame] | 652 | CBS_len(&cbs) != 0) { |
| 653 | SSL_SESSION_free(session); |
| 654 | ssl3_send_alert(ssl, SSL3_AL_FATAL, SSL_AD_DECODE_ERROR); |
| 655 | OPENSSL_PUT_ERROR(SSL, SSL_R_DECODE_ERROR); |
| 656 | return 0; |
| 657 | } |
| 658 | |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 659 | session->ticket_age_add_valid = 1; |
Steven Valdez | 1e6f11a | 2016-07-27 11:10:52 -0400 | [diff] [blame] | 660 | session->not_resumable = 0; |
| 661 | |
Steven Valdez | a833c35 | 2016-11-01 13:39:36 -0400 | [diff] [blame] | 662 | if (ssl->ctx->new_session_cb != NULL && |
Steven Valdez | 1e6f11a | 2016-07-27 11:10:52 -0400 | [diff] [blame] | 663 | ssl->ctx->new_session_cb(ssl, session)) { |
| 664 | /* |new_session_cb|'s return value signals that it took ownership. */ |
| 665 | return 1; |
| 666 | } |
| 667 | |
| 668 | SSL_SESSION_free(session); |
| 669 | return 1; |
| 670 | } |
David Benjamin | 4fe3c90 | 2016-08-16 02:17:03 -0400 | [diff] [blame] | 671 | |
| 672 | void ssl_clear_tls13_state(SSL *ssl) { |
David Benjamin | c8b6b4f | 2016-09-08 23:47:48 -0400 | [diff] [blame] | 673 | SSL_ECDH_CTX_cleanup(&ssl->s3->hs->ecdh_ctx); |
David Benjamin | 4fe3c90 | 2016-08-16 02:17:03 -0400 | [diff] [blame] | 674 | |
| 675 | OPENSSL_free(ssl->s3->hs->key_share_bytes); |
| 676 | ssl->s3->hs->key_share_bytes = NULL; |
| 677 | ssl->s3->hs->key_share_bytes_len = 0; |
| 678 | } |