Skip to content

Commit f1c918c

Browse files
committed
Support DTLS1.3 downgrade when server supports CID
With --enable-dtlscid, a client sending a Client Hello to a DLTS1.2 server that supports CID, the server provides the appropriate CID and assumes that CID has been negotiated. However, in the case of MbedTLS, it then rejects packets that do not match its expected CID from the client - as wolfSSL no longer sends the CID as it is not DTLS1.2. https://datatracker.ietf.org/doc/html/rfc9147#section-4 If a Connection ID is negotiated, then it MUST be contained in all datagrams. This fix drops the CID if a Hello Verify Request is received, so the second Client Hello does not include the CID. https://datatracker.ietf.org/doc/html/rfc6347#section-4.2.1 When responding to a HelloVerifyRequest, the client MUST use the same parameter values (version, random, session_id, cipher_suites, compression_method) as it did in the original ClientHello. Dropping the CID extension does not violate this.
1 parent 54370cc commit f1c918c

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

src/internal.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29593,6 +29593,10 @@ static int HashSkeData(WOLFSSL* ssl, enum wc_HashType hashType,
2959329593
#ifdef WOLFSSL_DTLS
2959429594
if (ssl->options.dtls) {
2959529595
DtlsMsgPoolReset(ssl);
29596+
#ifdef WOLFSSL_DTLS_CID
29597+
if (ssl->options.useDtlsCID)
29598+
DtlsCIDOnExtensionsParsed(ssl);
29599+
#endif /* WOLFSSL_DTLS_CID */
2959629600
}
2959729601
#endif
2959829602

0 commit comments

Comments
 (0)