Message224443
Thanks for the reminder Mark.
Yes, it is probably still an issue with the latest 2.7 release.
There were actually two issues:
- send send()/sendall() call didn't block because the test doesn't write enough data: we have since added a SOCK_MAX_SIZE constant to test_support just for that purpose, and the test has been updated, so it should now block (and succeed).
- the crash occurs because the test doesn't reset the alarm (with alarm(0)) upon exit, so if the alarm didn't go off during the test, it comes later when the original signal handler has been reset, and the default handler just exits.
To sum up, the first point is already fixed, and for the second point, here's a patch.
Note that 3.4 and default aren't affected, because the alarm is properly reset. |
|
Date |
User |
Action |
Args |
2014-07-31 19:45:10 | neologix | set | recipients:
+ neologix, pitrou, BreamoreBoy, erob, ddvento@ucar.edu |
2014-07-31 19:45:10 | neologix | set | messageid: <1406835910.5.0.434767754789.issue17085@psf.upfronthosting.co.za> |
2014-07-31 19:45:10 | neologix | link | issue17085 messages |
2014-07-31 19:45:10 | neologix | create | |
|