Skip to content

Commit cbb5b84

Browse files
committed
Merged in [8209] from rjsparks@nostrum.com:\n Fixed several places where files were not being closed.
- Legacy-Id: 8315 Note: SVN reference [8209] has been migrated to Git commit 4aae3ff
1 parent e661231 commit cbb5b84

3 files changed

Lines changed: 8 additions & 2 deletions

File tree

ietf/iesg/agenda.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import codecs
44
import datetime
55
from collections import OrderedDict
6+
from contextlib import closing
67

78
from django.conf import settings
89
from django.http import Http404
@@ -140,7 +141,7 @@ def fill_in_agenda_administrivia(date, sections):
140141

141142
for s, key, filename in extra_info_files:
142143
try:
143-
with codecs.open(filename, 'r', 'utf-8', 'replace') as f:
144+
with closing(codecs.open(filename, 'r', 'utf-8', 'replace')) as f:
144145
t = f.read().strip()
145146
except IOError:
146147
t = u"(Error reading %s)" % filename

ietf/ipr/search.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def patent_file_search(url, q):
3535
#print "*** Found file", fpath
3636
file = codecs.open(fpath, mode='r', encoding='utf-8', errors='replace')
3737
text = file.read()
38-
file.close
38+
file.close()
3939
return q in text
4040
return False
4141

ietf/meeting/views.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -401,6 +401,7 @@ def convert_to_pdf(doc_name):
401401
return
402402

403403
t,tempname = mkstemp()
404+
os.close(t)
404405
tempfile = open(tempname, "w")
405406

406407
pageend = 0;
@@ -431,6 +432,7 @@ def convert_to_pdf(doc_name):
431432
infile.close()
432433
tempfile.close()
433434
t,psname = mkstemp()
435+
os.close(t)
434436
pipe("enscript --margins 76::76: -B -q -p "+psname + " " +tempname)
435437
os.unlink(tempname)
436438
pipe("ps2pdf "+psname+" "+outpath)
@@ -473,6 +475,7 @@ def session_draft_tarfile(request, num, session):
473475
response['Content-Disposition'] = 'attachment; filename=%s-drafts.tgz'%(session)
474476
tarstream = tarfile.open('','w:gz',response)
475477
mfh, mfn = mkstemp()
478+
os.close(mfh)
476479
manifest = open(mfn, "w")
477480

478481
for doc_name in drafts:
@@ -512,6 +515,7 @@ def session_draft_pdf(request, num, session):
512515
drafts = session_draft_list(num, session);
513516
curr_page = 1
514517
pmh, pmn = mkstemp()
518+
os.close(pmh)
515519
pdfmarks = open(pmn, "w")
516520
pdf_list = ""
517521

@@ -528,6 +532,7 @@ def session_draft_pdf(request, num, session):
528532

529533
pdfmarks.close()
530534
pdfh, pdfn = mkstemp()
535+
os.close(pdfh)
531536
pipe("gs -dBATCH -dNOPAUSE -q -sDEVICE=pdfwrite -sOutputFile=" + pdfn + " " + pdf_list + " " + pmn)
532537

533538
pdf = open(pdfn,"r")

0 commit comments

Comments
 (0)