homepage

This issue tracker has been migrated to GitHub , and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients vstinner
Date 2019年06月26日.22:39:17
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1561588758.0.0.133123257973.issue37421@roundup.psfhosted.org>
In-reply-to
Content
Test leaking a temporary file in test_urllib:
test.test_urllib.urlretrieve_HttpTests.test_short_content_raises_ContentTooShortError_without_reporthook
I'm not sure of my fix:
diff --git a/Lib/urllib/request.py b/Lib/urllib/request.py
index f6ce9cb6d5..fdddd6e2d8 100644
--- a/Lib/urllib/request.py
+++ b/Lib/urllib/request.py
@@ -283,6 +283,7 @@ def urlretrieve(url, filename=None, reporthook=None, data=None):
 reporthook(blocknum, bs, size)
 
 if size >= 0 and read < size:
+ urlcleanup()
 raise ContentTooShortError(
 "retrieval incomplete: got only %i out of %i bytes"
 % (read, size), result)
History
Date User Action Args
2019年06月26日 22:39:18vstinnersetrecipients: + vstinner
2019年06月26日 22:39:18vstinnersetmessageid: <1561588758.0.0.133123257973.issue37421@roundup.psfhosted.org>
2019年06月26日 22:39:17vstinnerlinkissue37421 messages
2019年06月26日 22:39:17vstinnercreate

AltStyle によって変換されたページ (->オリジナル) /