Deal with no subject_confirmation element present.

Changed version to 2.0.0, needed by pysaml2 dependent projects.
IdentityPython · Mar 21, 2014 · 30b613e · 30b613e
1 parent 09f19c7
commit 30b613e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/setup.py b/setup.py
@@ -66,7 +66,7 @@ def run_tests(self):
 
 setup(
     name='pysaml2',
-    version='2.0.0beta',
+    version='2.0.0',
     description='Python implementation of SAML Version 2 to be used in a WSGI environment',
     # long_description = read("README"),
     author='Roland Hedberg',

diff --git a/src/saml2/response.py b/src/saml2/response.py
@@ -873,7 +873,9 @@ def verify_attesting_entity(self, address):
 
         correct = 0
         for subject_conf in self.assertion.subject.subject_confirmation:
-            if subject_conf.subject_confirmation_data.address:
+            if subject_conf.subject_confirmation_data is None:
+                correct += 1  # In reality undefined
+            elif subject_conf.subject_confirmation_data.address:
                 if subject_conf.subject_confirmation_data.address == address:
                     correct += 1
             else: