Issue 27811: _PyGen_Finalize() should not fail with an exception (original) (raw)
Created on 2016-08-20 09:35 by arigo, last changed 2022-04-11 14:58 by admin. This issue is now closed.
Files | |||
---|---|---|---|
File name | Uploaded | Description | Edit |
patch1.diff | arigo,2016-08-20 09:35 | review |
Messages (4) | ||
---|---|---|
msg273199 - (view) | Author: Armin Rigo (arigo) * ![]() |
Date: 2016-08-20 09:35 |
_PyGen_Finalize() should not fail with an exception. Doing so can cause various SystemErrors or even fatal errors. For example, run this with "python -Werror": import gc async def f(): pass f() gc.collect() # RuntimeWarning, but with -Werror Patch attached. Test missing (turn the above example in a test; also I didn't re-run all tests with this patch). | ||
msg274407 - (view) | Author: Benjamin Peterson (benjamin.peterson) * ![]() |
Date: 2016-09-05 17:16 |
Thank you, Armin. | ||
msg274408 - (view) | Author: Roundup Robot (python-dev) ![]() |
Date: 2016-09-05 17:17 |
New changeset 4d531711cbc7 by Benjamin Peterson in branch '3.5': do not allow _PyGen_Finalize to fail (closes #27811) https://hg.python.org/cpython/rev/4d531711cbc7 New changeset 1b14dbe9e98b by Benjamin Peterson in branch 'default': merge 3.5 (closes #27811) https://hg.python.org/cpython/rev/1b14dbe9e98b | ||
msg274428 - (view) | Author: Armin Rigo (arigo) * ![]() |
Date: 2016-09-05 20:22 |
You're welcome. Unrelated, but I'm collecting similar issues in a file at http://bitbucket.org/pypy/extradoc/raw/extradoc/planning/py3.5/cpython-crashers.rst . After reporting the first two, I stopped, and will report them all in bulk some time later, but you may be interested in fixing a couple more for the upcoming 3.6 release. |
History | |||
---|---|---|---|
Date | User | Action | Args |
2022-04-11 14:58:35 | admin | set | github: 71998 |
2016-09-05 20:22:58 | arigo | set | messages: + |
2016-09-05 17:17:51 | python-dev | set | status: open -> closednosy: + python-devmessages: + resolution: fixedstage: resolved |
2016-09-05 17:16:43 | benjamin.peterson | set | nosy: + benjamin.petersonmessages: + |
2016-08-20 09:35:05 | arigo | create |