Don't assume a decode error if session tlsext_ecpointformatlist is not NULL:

it can be legitimately set elsewhere.
This commit is contained in:
Dr. Stephen Henson
2010-11-16 22:41:07 +00:00
parent 5e7c9519b8
commit 4385b556b4

View File

@@ -779,8 +779,8 @@ int ssl_parse_clienthello_tlsext(SSL *s, unsigned char **p, unsigned char *d, in
{ {
if(s->session->tlsext_ecpointformatlist) if(s->session->tlsext_ecpointformatlist)
{ {
*al = TLS1_AD_DECODE_ERROR; OPENSSL_free(s->session->tlsext_ecpointformatlist);
return 0; s->session->tlsext_ecpointformatlist = NULL;
} }
s->session->tlsext_ecpointformatlist_length = 0; s->session->tlsext_ecpointformatlist_length = 0;
if ((s->session->tlsext_ecpointformatlist = OPENSSL_malloc(ecpointformatlist_length)) == NULL) if ((s->session->tlsext_ecpointformatlist = OPENSSL_malloc(ecpointformatlist_length)) == NULL)