Remove some redundant assignments
authorMatt Caswell <matt@openssl.org>
Fri, 6 Nov 2015 09:54:39 +0000 (09:54 +0000)
committerMatt Caswell <matt@openssl.org>
Fri, 6 Nov 2015 15:32:44 +0000 (15:32 +0000)
We were setting |s->renegotiate| and |s->new_session| to 0 twice in
tls_finish_handshake. This is redundant so now we just do it once!

Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
ssl/statem/statem_lib.c

index b0df7d2..2f13e92 100644 (file)
@@ -405,9 +405,6 @@ WORK_STATE tls_finish_handshake(SSL *s, WORK_STATE wst)
         s->new_session = 0;
 
         if (s->server) {
-            s->renegotiate = 0;
-            s->new_session = 0;
-
             ssl_update_cache(s, SSL_SESS_CACHE_SERVER);
 
             s->ctx->stats.sess_accept_good++;