MINOR: quic: Comment fix about the token found in Initial packets

Mention that the token is sent only by servers in both server and listener
packet parsers.
Remove a "TO DO" section in listener packet parser because there is nothing
more to do in this function about the token
This commit is contained in:
Frédéric Lécaille 2022-01-25 21:21:56 +01:00 committed by Amaury Denoyelle
parent 4852101fd2
commit 055ee6c14b

View File

@ -3942,7 +3942,8 @@ static ssize_t qc_srv_pkt_rcv(unsigned char **buf, const unsigned char *end,
* So, a client which receives an Initial packet with a token
* MUST discard the packet or generate a connection error with
* PROTOCOL_VIOLATION as type.
* The token must be provided in a Retry packet or NEW_TOKEN frame.
* The token may be provided in a Retry packet or NEW_TOKEN frame
* only by the QUIC server.
*/
pkt->token_len = token_len;
}
@ -4429,12 +4430,8 @@ static ssize_t qc_lstnr_pkt_rcv(unsigned char *buf, const unsigned char *end,
goto err;
}
/* XXX TO DO XXX 0 value means "the token is not present".
* A server which sends an Initial packet must not set the token.
* So, a client which receives an Initial packet with a token
* MUST discard the packet or generate a connection error with
* PROTOCOL_VIOLATION as type.
* The token must be provided in a Retry packet or NEW_TOKEN frame.
/* The token may be provided in a Retry packet or NEW_TOKEN frame
* only by the QUIC server.
*/
pkt->token_len = token_len;