Skip to content

Fix for TLS v1.2 session resumption with tickets where the server decides to do full handshake#8108

Closed
dgarske wants to merge 1 commit intowolfSSL:masterfrom
dgarske:tlsv12_sess_tick
Closed

Fix for TLS v1.2 session resumption with tickets where the server decides to do full handshake#8108
dgarske wants to merge 1 commit intowolfSSL:masterfrom
dgarske:tlsv12_sess_tick

Conversation

@dgarske
Copy link
Copy Markdown
Member

@dgarske dgarske commented Oct 23, 2024

Description

Fix for TLS v1.2 session resumption with tickets where the server decides to do a full handshake. The wrong sessionIDSz was being checked and should be the arrays one since it get set from the server_hello.

Fixes ZD 18822

Testing

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@dgarske dgarske self-assigned this Oct 23, 2024
@dgarske dgarske requested a review from julek-wolfssl October 23, 2024 21:58
julek-wolfssl
julek-wolfssl previously approved these changes Oct 23, 2024
Comment thread src/internal.c Outdated
…ides to do a full handshake. The wrong sessionIDSz was being checked and should be the arrays one since it get set from the server_hello.
@dgarske
Copy link
Copy Markdown
Member Author

dgarske commented Oct 23, 2024

Applied commit on top of #8097. That also resolves an issue with the server_hello session id being populated when it should not. Resolves the 964: test_session_ticket_no_id issue.

@dgarske dgarske closed this Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For This Release Release version 5.9.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants