summaryrefslogtreecommitdiffstats
path: root/ssl
diff options
context:
space:
mode:
authorMatt Caswell <matt@openssl.org>2017-02-08 17:27:09 +0000
committerMatt Caswell <matt@openssl.org>2017-02-14 13:14:25 +0000
commit429ff318d613047cf94accdc17e8d7c0dc144657 (patch)
treea7cb693b9e1d6faf53ed8d1e364158770892ac5d /ssl
parent319a33d0060b77d9446894b8a386718abcaee1a4 (diff)
Remove a double call to ssl3_send_alert()
Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/2341)
Diffstat (limited to 'ssl')
-rw-r--r--ssl/statem/statem_srvr.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/ssl/statem/statem_srvr.c b/ssl/statem/statem_srvr.c
index 66743c37ad..93ba63e650 100644
--- a/ssl/statem/statem_srvr.c
+++ b/ssl/statem/statem_srvr.c
@@ -3638,7 +3638,7 @@ STACK_OF(SSL_CIPHER) *ssl_bytes_to_cipher_list(SSL *s,
static int tls_construct_hello_retry_request(SSL *s, WPACKET *pkt)
{
- int al;
+ int al = SSL_AD_INTERNAL_ERROR;
/*
* TODO(TLS1.3): Remove the DRAFT version before release
@@ -3647,7 +3647,6 @@ static int tls_construct_hello_retry_request(SSL *s, WPACKET *pkt)
if (!WPACKET_put_bytes_u16(pkt, TLS1_3_VERSION_DRAFT)
|| !tls_construct_extensions(s, pkt, EXT_TLS1_3_HELLO_RETRY_REQUEST,
NULL, 0, &al)) {
- ssl3_send_alert(s, SSL3_AL_FATAL, al);
SSLerr(SSL_F_TLS_CONSTRUCT_HELLO_RETRY_REQUEST, ERR_R_INTERNAL_ERROR);
ssl3_send_alert(s, SSL3_AL_FATAL, al);
return 0;