From: Dr. Stephen Henson <steve@openssl.org>
Date: Thu, 17 Dec 2009 15:42:25 +0000 (+0000)
Subject: Alert to use is now defined in spec: update code
X-Git-Tag: OpenSSL_0_9_8m-beta1~27
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=98809a14586612a42f66324bde86c3c2e74f27e4;p=oweals%2Fopenssl.git

Alert to use is now defined in spec: update code
---

diff --git a/ssl/t1_lib.c b/ssl/t1_lib.c
index 07a3900c7f..8625b57104 100644
--- a/ssl/t1_lib.c
+++ b/ssl/t1_lib.c
@@ -589,8 +589,7 @@ int ssl_parse_clienthello_tlsext(SSL *s, unsigned char **p, unsigned char *d, in
 	if (!renegotiate_seen && s->new_session &&
 		!(s->options & SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION))
 		{
-		/* FIXME: Spec currently doesn't give alert to use */
-		*al = SSL_AD_ILLEGAL_PARAMETER;
+		*al = SSL_AD_HANDSHAKE_FAILURE;
 	 	SSLerr(SSL_F_SSL_PARSE_CLIENTHELLO_TLSEXT,
 				SSL_R_UNSAFE_LEGACY_RENEGOTIATION_DISABLED);
 		return 0;
@@ -709,8 +708,7 @@ int ssl_parse_serverhello_tlsext(SSL *s, unsigned char **p, unsigned char *d, in
 		(s->new_session || !(s->options & SSL_OP_LEGACY_SERVER_CONNECT))
 		&& !(s->options & SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION))
 		{
-		/* FIXME: Spec currently doesn't give alert to use */
-		*al = SSL_AD_ILLEGAL_PARAMETER;
+		*al = SSL_AD_HANDSHAKE_FAILURE;
 		SSLerr(SSL_F_SSL_PARSE_SERVERHELLO_TLSEXT,
 				SSL_R_UNSAFE_LEGACY_RENEGOTIATION_DISABLED);
 		return 0;