diff options
author | Alessandro Ghedini <alessandro@ghedini.me> | 2016-09-14 00:51:02 +0100 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2016-09-21 10:39:33 -0400 |
commit | 64d1cc09f912fdc92b12588cc9ad189978051a3e (patch) | |
tree | 34b2f13f9a628b745b4487119184a3021791ad90 /ssl | |
parent | 6bdd2637daa862706598d04ae2750a920ebee0ae (diff) |
Use switch instead of multiple ifs
Makes the logic a little bit clearer.
Reviewed-by: Andy Polyakov <appro@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/1571)
(cherry picked from commit 4f8a5f4da94bb70e06cf709beb1ddfa57a218c3d)
Diffstat (limited to 'ssl')
-rw-r--r-- | ssl/statem/statem.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/ssl/statem/statem.c b/ssl/statem/statem.c index 3df4ce61e1..df3008575d 100644 --- a/ssl/statem/statem.c +++ b/ssl/statem/statem.c @@ -566,22 +566,24 @@ static SUB_STATE_RETURN read_state_machine(SSL *s) /* Discard the packet data */ s->init_num = 0; - if (ret == MSG_PROCESS_ERROR) { + switch (ret) { + case MSG_PROCESS_ERROR: return SUB_STATE_ERROR; - } - if (ret == MSG_PROCESS_FINISHED_READING) { + case MSG_PROCESS_FINISHED_READING: if (SSL_IS_DTLS(s)) { dtls1_stop_timer(s); } return SUB_STATE_FINISHED; - } - if (ret == MSG_PROCESS_CONTINUE_PROCESSING) { + case MSG_PROCESS_CONTINUE_PROCESSING: st->read_state = READ_STATE_POST_PROCESS; st->read_state_work = WORK_MORE_A; - } else { + break; + + default: st->read_state = READ_STATE_HEADER; + break; } break; |