Matt Caswell fb62e47c78 Don't send a warning alert in TLSv1.3 %!s(int64=6) %!d(string=hai) anos
..
README f0de395950 Add a state machine README %!s(int64=9) %!d(string=hai) anos
extensions.c fb62e47c78 Don't send a warning alert in TLSv1.3 %!s(int64=6) %!d(string=hai) anos
extensions_clnt.c 73cc84a132 Suport TLSv1.3 draft 28 %!s(int64=6) %!d(string=hai) anos
extensions_cust.c 6738bf1417 Update copyright year %!s(int64=6) %!d(string=hai) anos
extensions_srvr.c 1aac20f509 Fix no-ec in combination with no-dh %!s(int64=6) %!d(string=hai) anos
statem.c f20404fce9 Don't fail on an out-of-order CCS in DTLS %!s(int64=6) %!d(string=hai) anos
statem.h d4ef4fbf46 Fix a crash in SSLfatal due to invalid enc_write_ctx %!s(int64=6) %!d(string=hai) anos
statem_clnt.c 10bda8f8dd Reformulate the if condition in tls_process_new_session_ticket %!s(int64=6) %!d(string=hai) anos
statem_dtls.c a925e7dbf4 Don't memcpy the contents of an empty fragment %!s(int64=6) %!d(string=hai) anos
statem_lib.c 73cc84a132 Suport TLSv1.3 draft 28 %!s(int64=6) %!d(string=hai) anos
statem_locl.h 5d67110173 Don't calculate the Finished MAC twice %!s(int64=6) %!d(string=hai) anos
statem_srvr.c 6cf2dbd9fa Don't store the ticket nonce in the session %!s(int64=6) %!d(string=hai) anos

README

State Machine Design
====================

This file provides some guidance on the thinking behind the design of the
state machine code to aid future maintenance.

The state machine code replaces an older state machine present in OpenSSL
versions 1.0.2 and below. The new state machine has the following objectives:
- Remove duplication of state code between client and server
- Remove duplication of state code between TLS and DTLS
- Simplify transitions and bring the logic together in a single location
so that it is easier to validate
- Remove duplication of code between each of the message handling functions
- Receive a message first and then work out whether that is a valid
transition - not the other way around (the other way causes lots of issues
where we are expecting one type of message next but actually get something
else)
- Separate message flow state from handshake state (in order to better
understand each)
- message flow state = when to flush buffers; handling restarts in the
event of NBIO events; handling the common flow of steps for reading a
message and the common flow of steps for writing a message etc
- handshake state = what handshake message are we working on now
- Control complexity: only the state machine can change state: keep all
the state changes local to the state machine component

The message flow state machine is divided into a reading sub-state machine and a
writing sub-state machine. See the source comments in statem.c for a more
detailed description of the various states and transitions possible.

Conceptually the state machine component is designed as follows:

libssl
|
---------------------------|-----statem.h--------------------------------------
|
_______V____________________
| |
| statem.c |
| |
| Core state machine code |
|____________________________|
statem_locl.h ^ ^
_________| |_______
| |
_____________|____________ _____________|____________
| | | |
| statem_clnt.c | | statem_srvr.c |
| | | |
| TLS/DTLS client specific | | TLS/DTLS server specific |
| state machine code | | state machine code |
|__________________________| |__________________________|
| |_______________|__ |
| ________________| | |
| | | |
____________V_______V________ ________V______V_______________
| | | |
| statem_both.c | | statem_dtls.c |
| | | |
| Non core functions common | | Non core functions common to |
| to both servers and clients | | both DTLS servers and clients |
|_____________________________| |_______________________________|