Skip to content

make curl_easy_duphandle() handle HSTS correctly#12315

Closed
bagder wants to merge 3 commits into
masterfrom
bagder/hsts-duped
Closed

make curl_easy_duphandle() handle HSTS correctly#12315
bagder wants to merge 3 commits into
masterfrom
bagder/hsts-duped

Conversation

@bagder

@bagder bagder commented Nov 13, 2023

Copy link
Copy Markdown
Member

No description provided.

To make it work properly with curl_easy_duphandle(). This, because
duphandle duplicates the entire 'UserDefined' struct by plain copy while
'hstslist' is a linked curl_list of file names. This would lead to a
double-free when the second of the two involved easy handles were
closed.
@github-actions github-actions Bot added the tests label Nov 13, 2023
@bagder bagder closed this in 7cb0322 Nov 13, 2023
bagder added a commit that referenced this pull request Nov 13, 2023
To make it work properly with curl_easy_duphandle(). This, because
duphandle duplicates the entire 'UserDefined' struct by plain copy while
'hstslist' is a linked curl_list of file names. This would lead to a
double-free when the second of the two involved easy handles were
closed.

Closes #12315
@bagder bagder deleted the bagder/hsts-duped branch November 13, 2023 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant