Skip to content

ssl-session fixups #19602

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
bagder wants to merge 4 commits into from
Closed

ssl-session fixups #19602

bagder wants to merge 4 commits into from

Conversation

@bagder
Copy link
Member

@bagder bagder commented Nov 19, 2025 •
edited
Loading

  • allow tests to depend on the ssl-sessions feature
  • add test 777 to verify some bad input sessions
  • fix the free in tool_ssls_load (addresses tool_ssls: aovid a potential mistake #19588)
  • free the ssl_session file name properly

@github-actions github-actions bot added the tests label Nov 19, 2025
@bagder bagder mentioned this pull request Nov 19, 2025
Closed
@bagder bagder marked this pull request as ready for review November 19, 2025 08:42
@bagder bagder closed this in c255d2f Nov 19, 2025
@bagder bagder deleted the bagder/ssls-sessions branch November 19, 2025 11:59
vszakats added a commit to vszakats/curl that referenced this pull request Nov 20, 2025
@vszakats
src: move memdebug.h to be the last include
adc979a 
`memdebug.h` must be included last within each source. This
breaks when including it in a header, which ends up being
included in the middle of other headers, and `memdebug.h`
also ending up in the middle.

Follow-up to c255d2f curl#19602
@vszakats vszakats mentioned this pull request Nov 20, 2025
Closed
vszakats added a commit that referenced this pull request Nov 21, 2025
@vszakats
src: move memdebug.h to be the last include
fd23d95 
`memdebug.h` must be included last within each source. This breaks when
including it in a header, which ends up being included in the middle of
other headers, and `memdebug.h` also ending up in the middle of
includes.

Follow-up to c255d2f #19602

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

Reviewers

No reviews

Assignees

No one assigned

Labels

cmdline tool tests

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bagder