diff options
author | David Benjamin <davidben@chromium.org> | 2014-11-30 10:58:52 +0300 |
---|---|---|
committer | Adam Langley <agl@google.com> | 2014-12-02 22:35:38 +0300 |
commit | beb47022b08b12cf2d7b2f7fb96484814ed4a9ef (patch) | |
tree | c159539444d5fa2fe79010b32d7333b0c2251dec /ssl/d1_clnt.c | |
parent | 8c6fe45c2f678041f5150460a0efc5e9ce5cda8a (diff) |
Remove redundant s->server assignments in handshake.
It should be set correctly prior to entering the handshake. Don't mask bugs by
assigning it.
Change-Id: Ib9bca8fad68916b3b242aad8819e3760e59e777a
Reviewed-on: https://boringssl-review.googlesource.com/2443
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'ssl/d1_clnt.c')
-rw-r--r-- | ssl/d1_clnt.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ssl/d1_clnt.c b/ssl/d1_clnt.c index 88a6af47..f291549f 100644 --- a/ssl/d1_clnt.c +++ b/ssl/d1_clnt.c @@ -136,6 +136,8 @@ int dtls1_connect(SSL *s) int new_state,state,skip=0; assert(s->handshake_func == dtls1_connect); + assert(!s->server); + ERR_clear_error(); ERR_clear_system_error(); @@ -160,7 +162,6 @@ int dtls1_connect(SSL *s) case SSL_ST_CONNECT: case SSL_ST_BEFORE|SSL_ST_CONNECT: - s->server=0; if (cb != NULL) cb(s,SSL_CB_HANDSHAKE_START,1); if ((s->version & 0xff00 ) != (DTLS1_VERSION & 0xff00)) |