From 40088d8b8190a2a33828a769c23bf35de542c7dc Mon Sep 17 00:00:00 2001 From: "Dr. Stephen Henson" Date: Tue, 17 Sep 2013 18:10:37 +0100 Subject: [PATCH] DTLS version usage fixes. Make DTLS behave like TLS when negotiating version: record layer has DTLS 1.0, message version is 1.2. Tolerate different version numbers if version hasn't been negotiated yet. --- ssl/d1_pkt.c | 4 +--- ssl/s3_clnt.c | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/ssl/d1_pkt.c b/ssl/d1_pkt.c index 39587039aa..aefd85d0c3 100644 --- a/ssl/d1_pkt.c +++ b/ssl/d1_pkt.c @@ -1557,9 +1557,7 @@ int do_dtls1_write(SSL *s, int type, const unsigned char *buf, unsigned int len, * we haven't decided which version to use yet send back using * version 1.0 header: otherwise some clients will ignore it. */ - if (s->state == DTLS1_ST_SW_HELLO_VERIFY_REQUEST_B - && s->method->version == DTLS_ANY_VERSION - && s->client_version == DTLS1_VERSION) + if (s->method->version == DTLS_ANY_VERSION) { *(p++)=DTLS1_VERSION>>8; *(p++)=DTLS1_VERSION&0xff; diff --git a/ssl/s3_clnt.c b/ssl/s3_clnt.c index 24c180c4d7..ce7b36aa14 100644 --- a/ssl/s3_clnt.c +++ b/ssl/s3_clnt.c @@ -928,7 +928,7 @@ int ssl3_get_server_hello(SSL *s) /* Hello verify request and/or server hello version may not * match so set first packet if we're negotiating version. */ - if (s->method->version == DTLS_ANY_VERSION) + if (SSL_IS_DTLS(s)) s->first_packet = 1; n=s->method->ssl_get_message(s, -- 2.25.1