diff options
| author | David Benjamin <davidben@chromium.org> | 2014-11-24 03:23:49 +0300 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2014-12-02 22:31:00 +0300 |
| commit | e58a71b9b30ebbe543cd42c450763937d54c609a (patch) | |
| tree | 70ee1265b4f35c8a8b9496eab0461ccc32fea406 /ssl/d1_srvr.c | |
| parent | 0b145c29a397d56ece7557c0b9b597e28fb34b99 (diff) | |
Trim impossible state combinations.
SSL_ST_BEFORE is never standalone. As of upstream's
413c4f45ed0508d2242638696b7665f499d68265, SSL_ST_BEFORE is only ever set paired
with SSL_ST_ACCEPT or SSL_ST_CONNECT.
Conversely, SSL_ST_OK is never paired with SSL_ST_ACCEPT or SSL_ST_CONNECT. As
far as I can tell, this combination has never been possible.
Change-Id: Ifbc8f147be821026cf59f3d5038f0dbad3b0a1d2
Reviewed-on: https://boringssl-review.googlesource.com/2433
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'ssl/d1_srvr.c')
| -rw-r--r-- | ssl/d1_srvr.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/ssl/d1_srvr.c b/ssl/d1_srvr.c index 43cfefa6..17a9a89f 100644 --- a/ssl/d1_srvr.c +++ b/ssl/d1_srvr.c @@ -183,10 +183,8 @@ int dtls1_accept(SSL *s) s->renegotiate=1; /* s->state=SSL_ST_ACCEPT; */ - case SSL_ST_BEFORE: case SSL_ST_ACCEPT: case SSL_ST_BEFORE|SSL_ST_ACCEPT: - case SSL_ST_OK|SSL_ST_ACCEPT: s->server=1; if (cb != NULL) cb(s,SSL_CB_HANDSHAKE_START,1); |