}
bool
-gss_eap_radius_attr_provider::initFromExistingContext(const gss_eap_attr_ctx *manager,
+gss_eap_radius_attr_provider::initWithExistingContext(const gss_eap_attr_ctx *manager,
const gss_eap_attr_provider *ctx)
{
const gss_eap_radius_attr_provider *radius;
- if (!gss_eap_attr_provider::initFromExistingContext(manager, ctx))
+ if (!gss_eap_attr_provider::initWithExistingContext(manager, ctx))
return false;
radius = static_cast<const gss_eap_radius_attr_provider *>(ctx);
}
bool
-gss_eap_radius_attr_provider::initFromGssContext(const gss_eap_attr_ctx *manager,
+gss_eap_radius_attr_provider::initWithGssContext(const gss_eap_attr_ctx *manager,
const gss_cred_id_t gssCred,
const gss_ctx_id_t gssCtx)
{
- if (!gss_eap_attr_provider::initFromGssContext(manager, gssCred, gssCtx))
+ if (!gss_eap_attr_provider::initWithGssContext(manager, gssCred, gssCtx))
return false;
if (gssCtx != GSS_C_NO_CONTEXT) {
vpcopy = paircopyvp(vp);
if (vpcopy == NULL) {
pairfree(&dst);
- throw new std::bad_alloc;
- return NULL;
+ throw std::bad_alloc();
}
*pDst = vpcopy;
pDst = &vpcopy->next;
char *b64;
if (base64Encode(vp->vp_octets, vp->length, &b64) < 0)
- throw new std::bad_alloc;
+ throw std::bad_alloc();
obj.set("value", b64);
GSSEAP_FREE(b64);
if (da != NULL) {
vp = pairalloc(da);
} else {
- /* Assume unknown attributes are octet strings */
- vp = paircreate(attrid, PW_TYPE_OCTETS);
- }
- if (vp == NULL) {
- throw new std::bad_alloc;
- goto fail;
+ int type = base64Valid(value.string()) ?
+ PW_TYPE_OCTETS : PW_TYPE_STRING;
+ vp = paircreate(attrid, type);
}
+ if (vp == NULL)
+ throw std::bad_alloc();
switch (vp->type) {
case PW_TYPE_INTEGER:
goto fail;
const char *str = value.string();
- size_t len = strlen(str);
+ ssize_t len = strlen(str);
/* this optimization requires base64Decode only understand packed encoding */
if (len >= BASE64_EXPAND(MAX_STRING_LEN))
goto fail;
- /*
- * If the attribute is unknown, we don't know its syntax; assume
- * it is an octet string and, if that fails to decode, a string.
- */
len = base64Decode(str, vp->vp_octets);
- if (len < 0) {
- if (da == NULL) {
- assert(len < MAX_STRING_LEN);
- vp->length = len;
- memcpy(vp->vp_strvalue, str, len + 1);
- } else
- goto fail;
- } else {
- vp->length = len;
- vp->vp_octets[len] = '\0';
- }
+ if (len < 0)
+ goto fail;
+
+ vp->length = len;
break;
}
}