[tor-commits] [flashproxy/master] Fix error handling for key and password file reading.

[dcf@xxxxxxxxxxxxxx]

commit 70bb6cbc5837b1fcb11ac6f751350db469ac95df
Author: David Fifield <david@xxxxxxxxxxxxxxx>
Date:   Thu Sep 20 07:40:15 2012 -0700

    Fix error handling for key and password file reading.
    
    The finally code was trying to close a file that had not been
    successfully opened.
---
 facilitator/facilitator-email-poller |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/facilitator/facilitator-email-poller b/facilitator/facilitator-email-poller
index 77f81e0..e3f3053 100755
--- a/facilitator/facilitator-email-poller
+++ b/facilitator/facilitator-email-poller
@@ -155,12 +155,12 @@ if options.password_filename is None:
     sys.exit(1)
 try:
     password_file = open(options.password_filename)
-except OSError, e:
+except Exception, e:
     print >> sys.stderr, """\
 Failed to open password file "%s": %s.\
 """ % (options.password_filename, str(e))
     sys.exit(1)
-else:
+try:
     if not check_perms(password_file.fileno()):
         print >> sys.stderr, "Refusing to run with group- or world-readable password file. Try"
         print >> sys.stderr, "\tchmod 600 %s" % options.password_filename
@@ -175,12 +175,12 @@ if options.key_filename is None:
     sys.exit(1)
 try:
     key_file = open(options.key_filename)
-except OSError, e:
+except Exception, e:
     print >> sys.stderr, """\
 Failed to open private key file "%s": %s.\
 """ % (options.key_filename, str(e))
     sys.exit(1)
-else:
+try:
     if not check_perms(key_file.fileno()):
         print >> sys.stderr, "Refusing to run with group- or world-readable private key file. Try"
         print >> sys.stderr, "\tchmod 600 %s" % options.key_filename



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits