Skip to content

multi: Improve parameter check for curl_multi_remove_handle#5116

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

multi: Improve parameter check for curl_multi_remove_handle#5116
jay wants to merge 1 commit intocurl:masterfrom
jay:improve_multi_rm

Conversation

@jay
Copy link
Copy Markdown
Member

@jay jay commented Mar 17, 2020

  • If an easy handle is owned by a multi different from the one specified
    then return CURLM_BAD_EASY_HANDLE.

Prior to this change I assume user error could cause corruption.

Closes #xxxx

- If an easy handle is owned by a multi different from the one specified
  then return CURLM_BAD_EASY_HANDLE.

Prior to this change I assume user error could cause corruption.

Closes #xxxx
Copy link
Copy Markdown
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea!

@jay jay closed this in a268ad5 Mar 18, 2020
@jay jay deleted the improve_multi_rm branch March 18, 2020 07:33
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.

2 participants