for (unsigned int i = ATTR_TYPE_MIN; i <= ATTR_TYPE_MAX; i++) {
gss_eap_attr_provider *provider;
- provider = (gssEapAttrFactories[i])();
-
+ if (gssEapAttrFactories[i] != NULL) {
+ provider = (gssEapAttrFactories[i])();
+ } else {
+ provider = NULL;
+ }
+
m_providers[i] = provider;
-
- /* providers should fail in constructor */
- assert(m_providers[i] != NULL);
}
}
{
unsigned int i;
- for (i = ATTR_TYPE_MIN; i < ATTR_TYPE_LOCAL; i++) {
+ for (i = ATTR_TYPE_MIN; i < ATTR_TYPE_MAX; i++) {
if (bufferEqual(&gssEapAttrPrefixes[i], prefix))
return i;
}
const gss_buffer_t
gss_eap_attr_ctx::attributeTypeToPrefix(unsigned int type)
{
- if (type < ATTR_TYPE_MIN || type >= ATTR_TYPE_LOCAL)
+ if (type < ATTR_TYPE_MIN || type >= ATTR_TYPE_MAX)
return GSS_C_NO_BUFFER;
return &gssEapAttrPrefixes[type];
for (unsigned int i = ATTR_TYPE_MIN; i <= ATTR_TYPE_MAX; i++) {
gss_eap_attr_provider *provider = m_providers[i];
- ret = provider->initFromExistingContext(this, manager->m_providers[i]);
+ if (provider == NULL)
+ continue;
+
+ ret = provider->initFromExistingContext(this,
+ manager->m_providers[i]);
if (ret == false)
break;
}
for (unsigned int i = ATTR_TYPE_MIN; i <= ATTR_TYPE_MAX; i++) {
gss_eap_attr_provider *provider = m_providers[i];
+ if (provider == NULL)
+ continue;
+
ret = provider->initFromGssContext(this, cred, ctx);
if (ret == false)
break;
decomposeAttributeName(attr, &type, &suffix);
provider = m_providers[type];
-
- provider->setAttribute(complete,
- (type == ATTR_TYPE_LOCAL) ? attr : &suffix,
- value);
+ if (provider != NULL) {
+ provider->setAttribute(complete,
+ (type == ATTR_TYPE_LOCAL) ? attr : &suffix,
+ value);
+ } else {
+ /* XXX TODO throw exception */
+ }
}
/*
decomposeAttributeName(attr, &type, &suffix);
provider = m_providers[type];
- if (provider != NULL) {
+ if (provider != NULL)
provider->deleteAttribute(type == ATTR_TYPE_LOCAL ? attr : &suffix);
- }
}
/*
size_t i;
for (i = ATTR_TYPE_MIN; i <= ATTR_TYPE_MAX; i++) {
- ret = m_providers[i]->getAttributeTypes(cb, data);
+ gss_eap_attr_provider *provider = m_providers[i];
+
+ if (provider == NULL)
+ continue;
+
+ ret = provider->getAttributeTypes(cb, data);
if (ret == false)
break;
}
args.attrs = *attrs;
for (i = ATTR_TYPE_MIN; i <= ATTR_TYPE_MAX; i++) {
+ gss_eap_attr_provider *provider = m_providers[i];
+
args.type = i;
- ret = m_providers[i]->getAttributeTypes(addAttribute, (void *)&args);
+ if (provider == NULL)
+ continue;
+
+ ret = provider->getAttributeTypes(addAttribute, (void *)&args);
if (ret == false)
break;
}
decomposeAttributeName(attr, &type, &suffix);
provider = m_providers[type];
+ if (provider == NULL)
+ return false;
ret = provider->getAttribute(type == ATTR_TYPE_LOCAL ? attr : &suffix,
authenticated, complete,
decomposeAttributeName(type_id, &type, &suffix);
provider = m_providers[type];
+ if (provider == NULL)
+ return (gss_any_t)NULL;
return provider->mapToAny(authenticated, &suffix);
}
decomposeAttributeName(type_id, &type, &suffix);
provider = m_providers[type];
-
- provider->releaseAnyNameMapping(&suffix, input);
+ if (provider != NULL)
+ provider->releaseAnyNameMapping(&suffix, input);
}
/*
time_t expiryTime = 0;
for (i = ATTR_TYPE_MIN; i <= ATTR_TYPE_MAX; i++) {
- time_t providerExpiryTime = m_providers[i]->getExpiryTime();
+ gss_eap_attr_provider *provider = m_providers[i];
+ time_t providerExpiryTime;
+
+ if (provider == NULL)
+ continue;
+ providerExpiryTime = provider->getExpiryTime();
if (providerExpiryTime == 0)
continue;