From c2fd15f6a7fb0d7a5944c5af0c01de678b271c90 Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Tue, 14 Feb 2017 11:48:24 +0000 Subject: [PATCH] Fix a shadowed global variable warning Reviewed-by: Rich Salz (Merged from https://github.com/openssl/openssl/pull/2609) --- ssl/ssl_locl.h | 2 +- ssl/tls13_enc.c | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/ssl/ssl_locl.h b/ssl/ssl_locl.h index cd948bd567..991766f05b 100644 --- a/ssl/ssl_locl.h +++ b/ssl/ssl_locl.h @@ -2167,7 +2167,7 @@ __owur int tls13_setup_key_block(SSL *s); __owur size_t tls13_final_finish_mac(SSL *s, const char *str, size_t slen, unsigned char *p); __owur int tls13_change_cipher_state(SSL *s, int which); -__owur int tls13_update_key(SSL *s, int write); +__owur int tls13_update_key(SSL *s, int send); __owur int tls13_hkdf_expand(SSL *s, const EVP_MD *md, const unsigned char *secret, const unsigned char *label, size_t labellen, diff --git a/ssl/tls13_enc.c b/ssl/tls13_enc.c index 3d8562bbf9..6faa4e0838 100644 --- a/ssl/tls13_enc.c +++ b/ssl/tls13_enc.c @@ -242,7 +242,7 @@ int tls13_setup_key_block(SSL *s) return 1; } -static int derive_secret_key_and_iv(SSL *s, int write, +static int derive_secret_key_and_iv(SSL *s, int send, const unsigned char *insecret, const unsigned char *hash, const unsigned char *label, @@ -281,7 +281,7 @@ static int derive_secret_key_and_iv(SSL *s, int write, goto err; } - if (EVP_CipherInit_ex(ciph_ctx, ciph, NULL, NULL, NULL, write) <= 0 + if (EVP_CipherInit_ex(ciph_ctx, ciph, NULL, NULL, NULL, send) <= 0 || !EVP_CIPHER_CTX_ctrl(ciph_ctx, EVP_CTRL_AEAD_SET_IVLEN, ivlen, NULL) || (taglen != 0 && !EVP_CIPHER_CTX_ctrl(ciph_ctx, EVP_CTRL_AEAD_SET_TAG, taglen, NULL)) @@ -292,7 +292,7 @@ static int derive_secret_key_and_iv(SSL *s, int write, #ifdef OPENSSL_SSL_TRACE_CRYPTO if (s->msg_callback) { - int wh = write ? TLS1_RT_CRYPTO_WRITE : 0; + int wh = send ? TLS1_RT_CRYPTO_WRITE : 0; if (ciph->key_len) s->msg_callback(2, s->version, wh | TLS1_RT_CRYPTO_KEY, @@ -459,7 +459,7 @@ int tls13_change_cipher_state(SSL *s, int which) return ret; } -int tls13_update_key(SSL *s, int write) +int tls13_update_key(SSL *s, int send) { static const unsigned char application_traffic[] = "application traffic secret"; @@ -470,12 +470,12 @@ int tls13_update_key(SSL *s, int write) EVP_CIPHER_CTX *ciph_ctx; int ret = 0; - if (s->server == write) + if (s->server == send) insecret = s->server_app_traffic_secret; else insecret = s->client_app_traffic_secret; - if (write) { + if (send) { iv = s->write_iv; ciph_ctx = s->enc_write_ctx; RECORD_LAYER_reset_write_sequence(&s->rlayer); @@ -485,7 +485,7 @@ int tls13_update_key(SSL *s, int write) RECORD_LAYER_reset_read_sequence(&s->rlayer); } - if (!derive_secret_key_and_iv(s, write, insecret, NULL, application_traffic, + if (!derive_secret_key_and_iv(s, send, insecret, NULL, application_traffic, sizeof(application_traffic) - 1, secret, iv, ciph_ctx)) goto err; -- 2.25.1