Skip to content

bpo-38785: Prevent asyncio from crashing #17144

Merged
miss-islington merged 1 commit into
python:masterfrom
asvetlov:issue38785
Nov 13, 2019
Merged

bpo-38785: Prevent asyncio from crashing #17144
miss-islington merged 1 commit into
python:masterfrom
asvetlov:issue38785

Conversation

@asvetlov
Copy link
Copy Markdown
Contributor

@asvetlov asvetlov commented Nov 13, 2019 •
edited by miss-islington
Loading

if parent __init__ is not called from a constructor of object derived from asyncio.Future

https://bugs.python.org/issue38785

Automerge-Triggered-By: @asvetlov

@asvetlov
Prevent asyncio from crashing if parent __init__ is not called from a
cc9d90d 
constructor of object derived from asyncio.Future
@asvetlov asvetlov requested a review from 1st1 as a code owner November 13, 2019 14:20
@asvetlov asvetlov requested a review from vstinner November 13, 2019 14:21
vstinner
vstinner approved these changes Nov 13, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

Note: I would prefer to rename "ENSURE_FUTURE_ALIVE" to "ENSURE_FUTURE_INITIALIZED", but that can be done in a separated PR and it's unrelated to the issue :-)

I tested manually this PR: it does fix https://bugs.python.org/issue38785

Without this PR, the example crash.

With this PR, I get a warning an no crash:

vstinner@apu$ ./python scratch_15.py 
In tratata before
In tratata2 before
Exception in callback <TaskStepMethWrapper object at 0x7f3ccfb878c0>()
handle: <Handle <TaskStepMethWrapper object at 0x7f3ccfb878c0>()>
Traceback (most recent call last):
  File "/home/vstinner/python/master/Lib/asyncio/events.py", line 81, in _run
    self._context.run(self._callback, *self._args)
RuntimeError: Future object is not initialized.
In tratata after

@asvetlov
Copy link
Copy Markdown
Contributor Author

asvetlov commented Nov 13, 2019

Yes, RuntimeError was my intention.
CFuture doesn't crash, PyFuture raises the same exception instead of returning None for uninitialized future on fut.get_loop() call.

@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Nov 13, 2019

Sorry, I can't merge this PR. Reason: Base branch was modified. Review and try the merge again..

@miss-islington miss-islington merged commit dad6be5 into python:master Nov 13, 2019
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Nov 13, 2019

Thanks @asvetlov for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@asvetlov asvetlov deleted the issue38785 branch November 13, 2019 21:37
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Nov 13, 2019

GH-17147 is a backport of this pull request to the 3.8 branch.

@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Nov 13, 2019

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 13, 2019
@asvetlov @miss-islington
bpo-38785: Prevent asyncio from crashing (pythonGH-17144)
2d7ae3f 
if parent `__init__` is not called from a constructor of object derived from `asyncio.Future`

https://bugs.python.org/issue38785
(cherry picked from commit dad6be5)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
miss-islington added a commit that referenced this pull request Nov 13, 2019
@miss-islington @asvetlov
bpo-38785: Prevent asyncio from crashing (GH-17144)
87b4d39 
if parent `__init__` is not called from a constructor of object derived from `asyncio.Future`

https://bugs.python.org/issue38785
(cherry picked from commit dad6be5)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
miss-islington added a commit that referenced this pull request Nov 13, 2019
@miss-islington @asvetlov
bpo-38785: Prevent asyncio from crashing (GH-17144)
694c03f 
if parent `__init__` is not called from a constructor of object derived from `asyncio.Future`

https://bugs.python.org/issue38785
(cherry picked from commit dad6be5)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
@asvetlov
bpo-38785: Prevent asyncio from crashing (pythonGH-17144)
198fbda 
if parent `__init__` is not called from a constructor of object derived from `asyncio.Future`



https://bugs.python.org/issue38785
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
@asvetlov @shihai1991
bpo-38785: Prevent asyncio from crashing (pythonGH-17144)
c45f06c 
if parent `__init__` is not called from a constructor of object derived from `asyncio.Future`



https://bugs.python.org/issue38785
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinner vstinner vstinner approved these changes

@1st1 1st1 1st1 approved these changes

Assignees

No one assigned

Labels

topic-asyncio

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@asvetlov @miss-islington @bedevere-bot @vstinner @1st1 @the-knights-who-say-ni