8000 bpo-33674: asyncio: Fix SSLProtocol race by vstinner · Pull Request #7175 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

bpo-33674: asyncio: Fix SSLProtocol race #7175 8000

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

Merged
merged 2 commits into from
May 28, 2018
Merged

bpo-33674: asyncio: Fix SSLProtocol race #7175

merged 2 commits into from
May 28, 2018

Conversation

vstinner
Copy link
Member
@vstinner vstinner commented May 28, 2018

Fix a race condition in SSLProtocol.connection_made() of
asyncio.sslproto: start immediately the handshake instead of using
call_soon(). Previously, data_received() could be called before the
handshake started, causing the handshake to hang or fail.

https://bugs.python.org/issue33674

Fix a race condition in SSLProtocol.connection_made() of
asyncio.sslproto: start immediately the handshake instead of using
call_soon(). Previously, data_received() could be called before the
handshake started, causing the handshake to hang or fail.
Create self._handshake_timeout_handle before calling
_process_write_backlog().
1st1 added a commit to MagicStack/uvloop that referenced this pull request May 28, 2018
1st1 added a commit to MagicStack/uvloop that referenced this pull request May 28, 2018
@vstinner vstinner merged commit be00a55 into python:master May 28, 2018
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7.
🐍🍒⛏🤖

@vstinner vstinner deleted the start_tls branch May 28, 2018 23:33
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 28, 2018
Fix a race condition in SSLProtocol.connection_made() of
asyncio.sslproto: start immediately the handshake instead of using
call_soon(). Previously, data_received() could be called before the
handshake started, causing the handshake to hang or fail.
(cherry picked from commit be00a55)

Co-authored-by: Victor Stinner <vstinner@redhat.com>
@bedevere-bot
Copy link

GH-7187 is a backport of this pull request to the 3.7 branch.

@miss-islington
Copy link
Contributor
miss-islington 8000 commented May 28, 2018

Sorry, @vstinner, I could not cleanly backport this to 3.6 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker be00a5583a2cb696335c527b921d1868266a42c6 3.6

@bedevere-bot
Copy link

GH-7188 is a backport of this pull request to the 3.6 branch.

ned-deily pushed a commit that referenced this pull request May 29, 2018
Fix a race condition in SSLProtocol.connection_made() of
asyncio.sslproto: start immediately the handshake instead of using
call_soon(). Previously, data_received() could be called before the
handshake started, causing the handshake to hang or fail.
(cherry picked from commit be00a55)

Co-authored-by: Victor Stinner <vstinner@redhat.com>
1st1 pushed a commit that referenced this pull request May 29, 2018
Fix a race condition in SSLProtocol.connection_made() of
asyncio.sslproto: start immediately the handshake instead of using
call_soon(). Previously, data_received() could be called before the
handshake started, causing the handshake to hang or fail.

(cherry picked from commit be00a55)
Olaf1022 pushed a commit to Olaf1022/Ultra-fast-asyncio that referenced this pull request Aug 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0