From be6a143eb5456c09c9c2828693e99ba60bcf213d Mon Sep 17 00:00:00 2001
From: Unknown <tolsen@limebits.com>
Date: Fri, 11 Jul 2008 12:29:18 -0400
Subject: actually set the result variable

---
 urllib2_kerberos.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/urllib2_kerberos.py b/urllib2_kerberos.py
index 7711a4e..fd7b9b1 100644
--- a/urllib2_kerberos.py
+++ b/urllib2_kerberos.py
@@ -97,11 +97,11 @@ class AbstractKerberosAuthHandler:
             log.critical("mutual auth failed. No negotiate header")
             return None
 
-        if k.authGSSClientStep(self.context, neg_value) < 1:
-            #TODO pyflakes flagged this so I commented it out --Gar
-            #
+        result = k.authGSSClientStep(self.context, neg_value)
+
+        if  result < 1:
             # this is a critical security warning
-            # uncommenting.  will change to a raise soon --Tim
+            # should change to a raise --Tim
             log.critical("mutual auth failed: authGSSClientStep returned result %d" % result)
             pass
 
-- 
cgit