Skip to content

multi: on socket callback error, remove socket hash entry nonetheless - #14557

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/multi-remove-transfer
Closed

bagder wants to merge 1 commit into
masterfrom
bagder/multi-remove-transfer

Conversation

@bagder

@bagder bagder commented Aug 15, 2024 •

Copy link
Copy Markdown
Member

Previously an error from the callback accidentally made libcurl keep the hash entry which would lead to the entry remaining and then potentially getting removed again which would lead to internal confusions.

This is an old issue (introduced in 2b3dd01), caught by the new asserts from c0233a3

Previously an error from the callback accidentally made libcurl keep the
hash entry which would lead to the entry remaining and then potentially
getting removed *again* which would lead to internal confusions.

This is an old issue, caught by the new asserts from c0233a3
@bagder bagder closed this in 160f023 Aug 15, 2024
@bagder
bagder deleted the bagder/multi-remove-transfer branch August 15, 2024 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant