diff -r 5b497ccfd4af Lib/http/server.py --- a/Lib/http/server.py Tue Jan 21 00:26:49 2014 -0500 +++ b/Lib/http/server.py Tue Jan 21 18:15:46 2014 +0200 @@ -678,8 +678,10 @@ """Serve a GET request.""" f = self.send_head() if f: - self.copyfile(f, self.wfile) - f.close() + try: + self.copyfile(f, self.wfile) + finally: + f.close() def do_HEAD(self): """Serve a HEAD request.""" @@ -720,13 +722,17 @@ except OSError: self.send_error(404, "File not found") return None - self.send_response(200) - self.send_header("Content-type", ctype) - fs = os.fstat(f.fileno()) - self.send_header("Content-Length", str(fs[6])) - self.send_header("Last-Modified", self.date_time_string(fs.st_mtime)) - self.end_headers() - return f + try: + self.send_response(200) + self.send_header("Content-type", ctype) + fs = os.fstat(f.fileno()) + self.send_header("Content-Length", str(fs[6])) + self.send_header("Last-Modified", self.date_time_string(fs.st_mtime)) + self.end_headers() + return f + except: + f.close() + raise def list_directory(self, path): """Helper to produce a directory listing (absent index.html). diff -r 5b497ccfd4af Lib/imghdr.py --- a/Lib/imghdr.py Tue Jan 21 00:26:49 2014 -0500 +++ b/Lib/imghdr.py Tue Jan 21 18:15:46 2014 +0200 @@ -7,18 +7,16 @@ #-------------------------# def what(file, h=None): - if h is None: - if isinstance(file, str): - f = open(file, 'rb') - h = f.read(32) - else: - location = file.tell() - h = file.read(32) - file.seek(location) - f = None - else: - f = None + f = None try: + if h is None: + if isinstance(file, str): + f = open(file, 'rb') + h = f.read(32) + else: + location = file.tell() + h = file.read(32) + file.seek(location) for tf in tests: res = tf(h, f) if res: diff -r 5b497ccfd4af Lib/mailcap.py --- a/Lib/mailcap.py Tue Jan 21 00:26:49 2014 -0500 +++ b/Lib/mailcap.py Tue Jan 21 18:15:46 2014 +0200 @@ -22,8 +22,8 @@ fp = open(mailcap, 'r') except OSError: continue - morecaps = readmailcapfile(fp) - fp.close() + with fp: + morecaps = readmailcapfile(fp) for key, value in morecaps.items(): if not key in caps: caps[key] = value diff -r 5b497ccfd4af Lib/mimetypes.py --- a/Lib/mimetypes.py Tue Jan 21 00:26:49 2014 -0500 +++ b/Lib/mimetypes.py Tue Jan 21 18:15:46 2014 +0200 @@ -363,9 +363,10 @@ f = open(file) except OSError: return None - db = MimeTypes() - db.readfp(f, True) - return db.types_map[True] + with f: + db = MimeTypes() + db.readfp(f, True) + return db.types_map[True] def _default_mime_types(): diff -r 5b497ccfd4af Lib/xml/etree/ElementInclude.py --- a/Lib/xml/etree/ElementInclude.py Tue Jan 21 00:26:49 2014 -0500 +++ b/Lib/xml/etree/ElementInclude.py Tue Jan 21 18:15:46 2014 +0200 @@ -76,14 +76,13 @@ def default_loader(href, parse, encoding=None): if parse == "xml": - file = open(href, 'rb') - data = ElementTree.parse(file).getroot() + with open(href, 'rb') as file: + data = ElementTree.parse(file).getroot() else: if not encoding: encoding = 'UTF-8' - file = open(href, 'r', encoding=encoding) - data = file.read() - file.close() + with open(href, 'r', encoding=encoding) as file: + data = file.read() return data ##