Skip to content

schannel: improve handshake procedure#18323

Closed
jay wants to merge 1 commit intocurl:masterfrom
jay:schannel_improve_step2
Closed

schannel: improve handshake procedure#18323
jay wants to merge 1 commit intocurl:masterfrom
jay:schannel_improve_step2

Conversation

@jay
Copy link
Member

@jay jay commented Aug 20, 2025

  • During handshake, do not require reading more data if unprocessed encrypted data that may be a complete TLS record is already available.

  • During handshake, check that the socket is writeable before processing encrypted data that may require an immediate reply to the server.

These two fixes are for issues that were found during renegotiation testing but could affect any handshake.

Prior to this change it was possible in some abnormal network conditions for the Schannel TLS handshake procedure to erroneously wait or error.

Ref: #18125

Closes #xxxxx

- During handshake, do not require reading more data if unprocessed
  encrypted data that may be a complete TLS record is already available.

- During handshake, check that the socket is writeable before processing
  encrypted data that may require an immediate reply to the server.

These two fixes are for issues that were found during renegotiation
testing but could affect any handshake.

Prior to this change it was possible in some abnormal network conditions
for the Schannel TLS handshake procedure to erroneously wait or error.

Ref: curl#18125

Closes #xxxxx
@jay jay added TLS Windows Windows-specific labels Aug 20, 2025
@jay jay closed this in b6a5f67 Aug 22, 2025
@jay jay deleted the schannel_improve_step2 branch August 22, 2025 05:56
jay added a commit to jay/curl that referenced this pull request Aug 27, 2025
Follow-up to b6a5f67 which improved the handshake procedure.

Ref: curl#18323

Closes #xxxx
jay added a commit that referenced this pull request Aug 27, 2025
Follow-up to b6a5f67 which improved the handshake procedure.

Ref: #18323

Closes #18410
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

TLS Windows Windows-specific

Development

Successfully merging this pull request may close these issues.

1 participant