Skip to content
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.

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

multi_wait: fix and improve Curl_poll error handling on Windows #9372

Closed
wants to merge 1 commit into from
Closed

multi_wait: fix and improve Curl_poll error handling on Windows #9372

wants to merge 1 commit into from

Conversation

mback2k
Copy link
Member

@mback2k mback2k commented Aug 25, 2022

First check for errors and return CURLM_UNRECOVERABLE_POLL
before moving forward and waiting on socket readiness events.

Reported-by: Daniel Stenberg
Ref: #9361

Follow up to #8961

@mback2k
multi_wait: fix and improve Curl_poll error handling on Windows
4ce56f0 
First check for errors and return CURLM_UNRECOVERABLE_POLL
before moving forward and waiting on socket readiness events.

Reported-by: Daniel Stenberg
Ref: curl#9361

Follow up to curl#8961
@mback2k mback2k added the Windows Windows-specific label Aug 25, 2022
@mback2k mback2k self-assigned this Aug 25, 2022
@mback2k mback2k mentioned this pull request Aug 25, 2022
Closed
@mback2k mback2k marked this pull request as ready for review August 26, 2022 09:36
@mback2k mback2k closed this in bc25c9e Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jay jay jay approved these changes

@MarcelRaad MarcelRaad MarcelRaad approved these changes

@bagder bagder Awaiting requested review from bagder

Assignees

@mback2k mback2k

Labels
Windows Windows-specific
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mback2k @jay @MarcelRaad