Message243042
A bit more detail on the patch-as-merged: it has all of Yury's new tests, but the actual bug turned out to just be a missing INCREF/DECREF pair in WITH_CLEANUP_START and WITH_CLEANUP_FINISH.
In the success case the reference counting errors cancelled each other out without causing a problem, as there was always a second live reference to the exception object on the stack.
However, in the case where the awaitable threw an exception the standard exception handling machinery took care of removing the saved exception from the stack, and correctly decremented the reference count, which then caused problems due to the missing INCREF in WITH_CLEANUP_START. |
|
Date |
User |
Action |
Args |
2015-05-13 06:06:46 | ncoghlan | set | recipients:
+ ncoghlan, gvanrossum, scoder, vstinner, asvetlov, python-dev, yselivanov |
2015-05-13 06:06:46 | ncoghlan | set | messageid: <1431497206.0.0.747143749738.issue24017@psf.upfronthosting.co.za> |
2015-05-13 06:06:45 | ncoghlan | link | issue24017 messages |
2015-05-13 06:06:45 | ncoghlan | create | |
|