summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjsing <>2020-10-07 08:43:34 +0000
committerjsing <>2020-10-07 08:43:34 +0000
commiteaae66413ad2b6a728338460fadaf9b311f4e851 (patch)
treea4dafa9c14aa253f898d3d6a2d8ad7c9d88b0ee1
parentdb49c1b46248450826f76c7d1a09328b0768d673 (diff)
downloadopenbsd-eaae66413ad2b6a728338460fadaf9b311f4e851.tar.gz
openbsd-eaae66413ad2b6a728338460fadaf9b311f4e851.tar.bz2
openbsd-eaae66413ad2b6a728338460fadaf9b311f4e851.zip
Mop up various things that are now unused with the new record layer.
ok inoguchi@ tb@
-rw-r--r--src/lib/libssl/ssl_locl.h15
-rw-r--r--src/lib/libssl/t1_enc.c15
-rw-r--r--src/lib/libssl/t1_lib.c8
3 files changed, 6 insertions, 32 deletions
diff --git a/src/lib/libssl/ssl_locl.h b/src/lib/libssl/ssl_locl.h
index 41b3c02724..b207dc65e9 100644
--- a/src/lib/libssl/ssl_locl.h
+++ b/src/lib/libssl/ssl_locl.h
@@ -1,4 +1,4 @@
1/* $OpenBSD: ssl_locl.h,v 1.298 2020/10/03 18:01:55 jsing Exp $ */ 1/* $OpenBSD: ssl_locl.h,v 1.299 2020/10/07 08:43:34 jsing Exp $ */
2/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) 2/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
3 * All rights reserved. 3 * All rights reserved.
4 * 4 *
@@ -317,10 +317,6 @@ __BEGIN_HIDDEN_DECLS
317#define SSL_IS_DTLS(s) \ 317#define SSL_IS_DTLS(s) \
318 (s->method->internal->version == DTLS1_VERSION) 318 (s->method->internal->version == DTLS1_VERSION)
319 319
320/* See if we need explicit IV. */
321#define SSL_USE_EXPLICIT_IV(s) \
322 (s->method->internal->ssl3_enc->enc_flags & SSL_ENC_FLAG_EXPLICIT_IV)
323
324/* See if we use signature algorithms extension. */ 320/* See if we use signature algorithms extension. */
325#define SSL_USE_SIGALGS(s) \ 321#define SSL_USE_SIGALGS(s) \
326 (s->method->internal->ssl3_enc->enc_flags & SSL_ENC_FLAG_SIGALGS) 322 (s->method->internal->ssl3_enc->enc_flags & SSL_ENC_FLAG_SIGALGS)
@@ -751,10 +747,6 @@ typedef struct ssl_internal_st {
751 747
752 STACK_OF(SSL_CIPHER) *cipher_list_tls13; 748 STACK_OF(SSL_CIPHER) *cipher_list_tls13;
753 749
754 /* These are the ones being used, the ones in SSL_SESSION are
755 * the ones to be 'copied' into these ones */
756 int mac_flags;
757
758 SSL_AEAD_CTX *aead_read_ctx; /* AEAD context. If non-NULL, then 750 SSL_AEAD_CTX *aead_read_ctx; /* AEAD context. If non-NULL, then
759 enc_read_ctx and read_hash are 751 enc_read_ctx and read_hash are
760 ignored. */ 752 ignored. */
@@ -842,8 +834,6 @@ typedef struct ssl3_state_internal_st {
842 int read_mac_secret_size; 834 int read_mac_secret_size;
843 unsigned char read_mac_secret[EVP_MAX_MD_SIZE]; 835 unsigned char read_mac_secret[EVP_MAX_MD_SIZE];
844 unsigned char write_sequence[SSL3_SEQUENCE_SIZE]; 836 unsigned char write_sequence[SSL3_SEQUENCE_SIZE];
845 int write_mac_secret_size;
846 unsigned char write_mac_secret[EVP_MAX_MD_SIZE];
847 837
848 SSL3_BUFFER_INTERNAL rbuf; /* read IO goes into here */ 838 SSL3_BUFFER_INTERNAL rbuf; /* read IO goes into here */
849 SSL3_BUFFER_INTERNAL wbuf; /* write IO goes into here */ 839 SSL3_BUFFER_INTERNAL wbuf; /* write IO goes into here */
@@ -1081,9 +1071,6 @@ typedef struct ssl3_enc_method {
1081 * Flag values for enc_flags. 1071 * Flag values for enc_flags.
1082 */ 1072 */
1083 1073
1084/* Uses explicit IV. */
1085#define SSL_ENC_FLAG_EXPLICIT_IV (1 << 0)
1086
1087/* Uses signature algorithms extension. */ 1074/* Uses signature algorithms extension. */
1088#define SSL_ENC_FLAG_SIGALGS (1 << 1) 1075#define SSL_ENC_FLAG_SIGALGS (1 << 1)
1089 1076
diff --git a/src/lib/libssl/t1_enc.c b/src/lib/libssl/t1_enc.c
index 7a71a08434..debbf286f6 100644
--- a/src/lib/libssl/t1_enc.c
+++ b/src/lib/libssl/t1_enc.c
@@ -1,4 +1,4 @@
1/* $OpenBSD: t1_enc.c,v 1.124 2020/10/03 17:35:16 jsing Exp $ */ 1/* $OpenBSD: t1_enc.c,v 1.125 2020/10/07 08:43:34 jsing Exp $ */
2/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) 2/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
3 * All rights reserved. 3 * All rights reserved.
4 * 4 *
@@ -423,11 +423,6 @@ tls1_change_cipher_state_cipher(SSL *s, char is_read,
423 stream_mac = S3I(s)->hs.new_cipher->algorithm2 & TLS1_STREAM_MAC; 423 stream_mac = S3I(s)->hs.new_cipher->algorithm2 & TLS1_STREAM_MAC;
424 424
425 if (is_read) { 425 if (is_read) {
426 if (stream_mac)
427 s->internal->mac_flags |= SSL_MAC_FLAG_READ_MAC_STREAM;
428 else
429 s->internal->mac_flags &= ~SSL_MAC_FLAG_READ_MAC_STREAM;
430
431 ssl_clear_cipher_read_state(s); 426 ssl_clear_cipher_read_state(s);
432 427
433 if ((cipher_ctx = EVP_CIPHER_CTX_new()) == NULL) 428 if ((cipher_ctx = EVP_CIPHER_CTX_new()) == NULL)
@@ -445,11 +440,6 @@ tls1_change_cipher_state_cipher(SSL *s, char is_read,
445 S3I(s)->read_mac_secret, mac_secret_size)) 440 S3I(s)->read_mac_secret, mac_secret_size))
446 goto err; 441 goto err;
447 } else { 442 } else {
448 if (stream_mac)
449 s->internal->mac_flags |= SSL_MAC_FLAG_WRITE_MAC_STREAM;
450 else
451 s->internal->mac_flags &= ~SSL_MAC_FLAG_WRITE_MAC_STREAM;
452
453 /* 443 /*
454 * DTLS fragments retain a pointer to the compression, cipher 444 * DTLS fragments retain a pointer to the compression, cipher
455 * and hash contexts, so that it can restore state in order 445 * and hash contexts, so that it can restore state in order
@@ -581,9 +571,6 @@ tls1_change_cipher_state(SSL *s, int which)
581 if (is_read) { 571 if (is_read) {
582 memcpy(S3I(s)->read_mac_secret, mac_secret, mac_secret_size); 572 memcpy(S3I(s)->read_mac_secret, mac_secret, mac_secret_size);
583 S3I(s)->read_mac_secret_size = mac_secret_size; 573 S3I(s)->read_mac_secret_size = mac_secret_size;
584 } else {
585 memcpy(S3I(s)->write_mac_secret, mac_secret, mac_secret_size);
586 S3I(s)->write_mac_secret_size = mac_secret_size;
587 } 574 }
588 575
589 if (aead != NULL) { 576 if (aead != NULL) {
diff --git a/src/lib/libssl/t1_lib.c b/src/lib/libssl/t1_lib.c
index f091dd001b..5635c8ff43 100644
--- a/src/lib/libssl/t1_lib.c
+++ b/src/lib/libssl/t1_lib.c
@@ -1,4 +1,4 @@
1/* $OpenBSD: t1_lib.c,v 1.176 2020/09/12 17:25:11 tb Exp $ */ 1/* $OpenBSD: t1_lib.c,v 1.177 2020/10/07 08:43:34 jsing Exp $ */
2/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) 2/* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com)
3 * All rights reserved. 3 * All rights reserved.
4 * 4 *
@@ -130,12 +130,12 @@ SSL3_ENC_METHOD TLSv1_enc_data = {
130}; 130};
131 131
132SSL3_ENC_METHOD TLSv1_1_enc_data = { 132SSL3_ENC_METHOD TLSv1_1_enc_data = {
133 .enc_flags = SSL_ENC_FLAG_EXPLICIT_IV, 133 .enc_flags = 0,
134}; 134};
135 135
136SSL3_ENC_METHOD TLSv1_2_enc_data = { 136SSL3_ENC_METHOD TLSv1_2_enc_data = {
137 .enc_flags = SSL_ENC_FLAG_EXPLICIT_IV|SSL_ENC_FLAG_SIGALGS| 137 .enc_flags = SSL_ENC_FLAG_SIGALGS|SSL_ENC_FLAG_SHA256_PRF|
138 SSL_ENC_FLAG_SHA256_PRF|SSL_ENC_FLAG_TLS1_2_CIPHERS, 138 SSL_ENC_FLAG_TLS1_2_CIPHERS,
139}; 139};
140 140
141int 141int