Alert to use is now defined in spec: update code
This commit is contained in:
parent
2d3855fc6e
commit
54bc369ad7
@ -971,8 +971,7 @@ int ssl_parse_clienthello_tlsext(SSL *s, unsigned char **p, unsigned char *d, in
|
|||||||
if (!renegotiate_seen && s->new_session &&
|
if (!renegotiate_seen && s->new_session &&
|
||||||
!(s->options & SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION))
|
!(s->options & SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION))
|
||||||
{
|
{
|
||||||
/* FIXME: Spec currently doesn't give alert to use */
|
*al = SSL_AD_HANDSHAKE_FAILURE;
|
||||||
*al = SSL_AD_ILLEGAL_PARAMETER;
|
|
||||||
SSLerr(SSL_F_SSL_PARSE_CLIENTHELLO_TLSEXT,
|
SSLerr(SSL_F_SSL_PARSE_CLIENTHELLO_TLSEXT,
|
||||||
SSL_R_UNSAFE_LEGACY_RENEGOTIATION_DISABLED);
|
SSL_R_UNSAFE_LEGACY_RENEGOTIATION_DISABLED);
|
||||||
return 0;
|
return 0;
|
||||||
@ -1161,8 +1160,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->new_session || !(s->options & SSL_OP_LEGACY_SERVER_CONNECT))
|
||||||
&& !(s->options & SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION))
|
&& !(s->options & SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION))
|
||||||
{
|
{
|
||||||
/* FIXME: Spec currently doesn't give alert to use */
|
*al = SSL_AD_HANDSHAKE_FAILURE;
|
||||||
*al = SSL_AD_ILLEGAL_PARAMETER;
|
|
||||||
SSLerr(SSL_F_SSL_PARSE_SERVERHELLO_TLSEXT,
|
SSLerr(SSL_F_SSL_PARSE_SERVERHELLO_TLSEXT,
|
||||||
SSL_R_UNSAFE_LEGACY_RENEGOTIATION_DISABLED);
|
SSL_R_UNSAFE_LEGACY_RENEGOTIATION_DISABLED);
|
||||||
return 0;
|
return 0;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user