commit 05ff9904010a488cc640637ac8255cae41b270dd
author R David Murray <rdmurray@bitdance.com> Fri Jun 17 12:54:56 2011 -0400
committer R David Murray <rdmurray@bitdance.com> Fri Jun 17 12:54:56 2011 -0400
tree 4dcec34c3a4b2c246bd0b299e96c0e93dae3b9c8
parent 26de69dff824a9474ea0044afcbff2d30f109e1e

#11767: use context manager to close file in __getitem__ to prevent FD leak

All of the other methods in mailbox that create message objects take care to
close the file descriptors they use, so it seems to make sense to have
__getitem__ do so as well.

Patch by Filip Gruszczyński.

Misc/NEWS

diff --git a/Misc/NEWS b/Misc/NEWS
index 6cc0f03..ec8b72e 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -25,6 +25,8 @@
 Library
 -------
 
+- Issue #11767: Correct file descriptor leak in mailbox's __getitem__ method.
+
 - Issue #12133: AbstractHTTPHandler.do_open() of urllib.request closes the HTTP
   connection if its getresponse() method fails with a socket error. Patch
   written by Ezio Melotti.