Skip to content

Commit

Permalink
fixup! Fix an assertion failure which happens when a DTLS 1.3 client …
Browse files Browse the repository at this point in the history
…receives a HelloVerifyRequest.
  • Loading branch information
fwh-dc committed Jun 4, 2024
1 parent 58c2354 commit 7eead5b
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions ssl/statem/statem_clnt.c
Original file line number Diff line number Diff line change
Expand Up @@ -1279,9 +1279,7 @@ CON_FUNC_RETURN tls_construct_client_hello(SSL_CONNECTION *s, WPACKET *pkt)
}

/* cookie stuff for DTLS */
if (SSL_CONNECTION_IS_DTLS(s)
&& ossl_assert(!(s->d1->hello_verify_request == SSL_HVR_RECEIVED)
|| s->d1->cookie_len > 0)) {
if (SSL_CONNECTION_IS_DTLS(s)) {
if (s->d1->cookie_len > sizeof(s->d1->cookie)
|| !WPACKET_sub_memcpy_u8(pkt, s->d1->cookie,
s->d1->cookie_len)) {
Expand Down

0 comments on commit 7eead5b

Please sign in to comment.