Don't crash if there are zero valued attributes
authorLuke Howard <lukeh@padl.com>
Sun, 3 Apr 2011 07:57:09 +0000 (17:57 +1000)
committerLuke Howard <lukeh@padl.com>
Sun, 3 Apr 2011 08:00:48 +0000 (18:00 +1000)
util_saml.cpp
util_shib.cpp

index fd84e44..26bf544 100644 (file)
@@ -662,7 +662,7 @@ gss_eap_saml_attr_provider::getAttribute(const gss_buffer_t attr,
 
     if (i == -1)
         i = 0;
-    else if (i >= nvalues)
+    if (i >= nvalues)
         return false;
 #ifdef __APPLE__
     av = (const saml2::AttributeValue *)((void *)(a->getAttributeValues().at(i)));
index 07ee06c..778ad25 100644 (file)
@@ -331,7 +331,7 @@ gss_eap_shib_attr_provider::getAttribute(const gss_buffer_t attr,
 
     if (i == -1)
         i = 0;
-    else if (i >= nvalues)
+    if (i >= nvalues)
         return false;
 
     buf.value = (void *)shibAttr->getSerializedValues()[*more].c_str();