Fix some build errors, and build with flat_namespace

[moonshot.git] / mech_eap / accept_sec_context.c

diff --git a/mech_eap/accept_sec_context.c b/mech_eap/accept_sec_context.c
index bb182bc..6def93b 100644 (file)
--- a/mech_eap/accept_sec_context.c
+++ b/mech_eap/accept_sec_context.c
@@ -48,10 +48,6 @@ acceptReady(OM_uint32 *minor, gss_ctx_id_t ctx, gss_cred_id_t cred)
     if (GSS_ERROR(major))
         return major;
 
-    /*
-     * Now, if we have a username from the identity packet, discard it
-     * because it's unauthenticated.
-     */
     gssEapReleaseName(&tmpMinor, &ctx->initiatorName);
 
     vp = rc_avpair_get(ctx->acceptorCtx.avps, PW_USER_NAME, 0);
@@ -70,8 +66,8 @@ acceptReady(OM_uint32 *minor, gss_ctx_id_t ctx, gss_cred_id_t cred)
     ctx->initiatorName->attrCtx = gssEapCreateAttrContext(cred, ctx);
 
     vp = rc_avpair_get(ctx->acceptorCtx.avps,
-                       RADIUS_VENDOR_ATTR_MS_MPPE_SEND_KEY,
-                       RADIUS_VENDOR_ID_MICROSOFT);
+                       VENDOR_ATTR_MS_MPPE_SEND_KEY,
+                       VENDOR_ID_MICROSOFT);
     if (ctx->encryptionType != ENCTYPE_NULL && vp != NULL) {
         major = gssEapDeriveRfc3961Key(minor,
                                        (unsigned char *)vp->strvalue,
@@ -130,10 +126,10 @@ eapGssSmAcceptIdentity(OM_uint32 *minor,
     if (GSS_ERROR(major))
         return major;
 
-    if (ctx->acceptorName == GSS_C_NO_NAME &&
-        cred != GSS_C_NO_CREDENTIAL &&
-        cred->name != GSS_C_NO_NAME) {
-        major = gss_duplicate_name(minor, cred->name, &ctx->acceptorName);
+    assert(ctx->acceptorName == GSS_C_NO_NAME);
+
+    if (cred != GSS_C_NO_CREDENTIAL && cred->name != GSS_C_NO_NAME) {
+        major = gssEapDuplicateName(minor, cred->name, &ctx->acceptorName);
         if (GSS_ERROR(major))
             return major;
     }
@@ -156,28 +152,55 @@ eapGssSmAcceptIdentity(OM_uint32 *minor,
 }
 
 static OM_uint32
-importInitiatorIdentity(OM_uint32 *minor,
-                        gss_ctx_id_t ctx,
-                        gss_buffer_t inputToken,
-                        gss_buffer_t nameBuf)
+setAcceptorIdentity(OM_uint32 *minor,
+                    gss_ctx_id_t ctx,
+                    VALUE_PAIR **avps)
 {
     OM_uint32 major, tmpMinor;
-    struct eap_hdr *pdu = (struct eap_hdr *)inputToken->value;
-    unsigned char *pos = (unsigned char *)(pdu + 1);
-    gss_name_t name;
+    gss_buffer_desc nameBuf;
+    krb5_principal krbPrinc;
+
+    /* Awaits further specification */
+    if (ctx->acceptorName == GSS_C_NO_NAME)
+        return GSS_S_COMPLETE;
+
+    krbPrinc = ctx->acceptorName->krbPrincipal;
+    assert(krbPrinc != NULL);
 
-    assert(pdu->code == EAP_CODE_RESPONSE);
-    assert(pos[0] == EAP_TYPE_IDENTITY);
+    if (krbPrinc->length < 2)
+        return GSS_S_BAD_NAME;
 
-    nameBuf->value = pos + 1;
-    nameBuf->length = inputToken->length - sizeof(*pdu) - 1;
+    nameBuf.value = krbPrinc->data[0].data;
+    nameBuf.length = krbPrinc->data[0].length;
 
-    major = gssEapImportName(minor, nameBuf, GSS_C_NT_USER_NAME, &name);
+    major = addAvpFromBuffer(minor, ctx->acceptorCtx.radHandle, avps,
+                             VENDOR_ATTR_GSS_ACCEPTOR_SERVICE_NAME,
+                             VENDOR_ID_UKERNA,
+                             &nameBuf);
     if (GSS_ERROR(major))
         return major;
 
-    gssEapReleaseName(&tmpMinor, &ctx->initiatorName);
-    ctx->initiatorName = name;
+    nameBuf.value = krbPrinc->data[1].data;
+    nameBuf.length = krbPrinc->data[2].length;
+
+    major = addAvpFromBuffer(minor, ctx->acceptorCtx.radHandle, avps,
+                             VENDOR_ATTR_GSS_ACCEPTOR_HOST_NAME,
+                             VENDOR_ID_UKERNA,
+                             &nameBuf);
+    if (GSS_ERROR(major))
+        return major;
+
+    if (krbPrinc->realm.data != NULL) {
+        nameBuf.value = krbPrinc->realm.data;
+        nameBuf.length = krbPrinc->realm.length;
+
+        major = addAvpFromBuffer(minor, ctx->acceptorCtx.radHandle, avps,
+                                 VENDOR_ATTR_GSS_ACCEPTOR_REALM_NAME,
+                                 VENDOR_ID_UKERNA,
+                                 &nameBuf);
+        if (GSS_ERROR(major))
+            return major;
+    }
 
     return GSS_S_COMPLETE;
 }
@@ -206,21 +229,25 @@ eapGssSmAcceptAuthenticate(OM_uint32 *minor,
     if (inputToken->length > sizeof(*pdu) &&
         pdu->code == EAP_CODE_RESPONSE &&
         pos[0] == EAP_TYPE_IDENTITY) {
-        major = importInitiatorIdentity(minor, ctx, inputToken, &nameBuf);
+        /*
+         * XXX TODO do we really need to set User-Name? FreeRADIUS does
+         * not appear to require it.
+         */
+        major = addAvpFromBuffer(minor, rh, &send, PW_USER_NAME, 0, &nameBuf);
         if (GSS_ERROR(major))
             goto cleanup;
 
-        major = addAvpFromBuffer(minor, rh, &send, PW_USER_NAME, &nameBuf);
+        major = setAcceptorIdentity(minor, ctx, &send);
         if (GSS_ERROR(major))
             goto cleanup;
     }
 
-    major = addAvpFromBuffer(minor, rh, &send, PW_EAP_MESSAGE, inputToken);
+    major = addAvpFromBuffer(minor, rh, &send, PW_EAP_MESSAGE, 0, inputToken);
     if (GSS_ERROR(major))
         goto cleanup;
 
     if (ctx->acceptorCtx.lastStatus == CHALLENGE_RC) {
-        major = addAvpFromBuffer(minor, rh, &send, PW_STATE,
+        major = addAvpFromBuffer(minor, rh, &send, PW_STATE, 0,
                                  &ctx->acceptorCtx.state);
         if (GSS_ERROR(major))
             goto cleanup;
@@ -250,14 +277,14 @@ eapGssSmAcceptAuthenticate(OM_uint32 *minor,
 
     ctx->acceptorCtx.lastStatus = code;
 
-    major = getBufferFromAvps(minor, received, PW_EAP_MESSAGE,
+    major = getBufferFromAvps(minor, received, PW_EAP_MESSAGE, 0,
                               outputToken, TRUE);
     if ((major == GSS_S_UNAVAILABLE && code != OK_RC) ||
         GSS_ERROR(major))
         goto cleanup;
 
     if (code == CHALLENGE_RC) {
-        major = getBufferFromAvps(minor, received, PW_STATE,
+        major = getBufferFromAvps(minor, received, PW_STATE, 0,
                                   &ctx->acceptorCtx.state, TRUE);
         if (major != GSS_S_UNAVAILABLE && GSS_ERROR(major))
             goto cleanup;
@@ -452,12 +479,12 @@ gss_accept_sec_context(OM_uint32 *minor,
 
     if (major == GSS_S_COMPLETE) {
         if (src_name != NULL && ctx->initiatorName != GSS_C_NO_NAME) {
-            major = gss_duplicate_name(&tmpMinor, ctx->initiatorName, src_name);
+            major = gssEapDuplicateName(&tmpMinor, ctx->initiatorName, src_name);
             if (GSS_ERROR(major))
                 goto cleanup;
         }
         if (time_rec != NULL)
-            gss_context_time(&tmpMinor, ctx, time_rec);
+            gssEapContextTime(&tmpMinor, ctx, time_rec);
     }
 
     assert(ctx->state == EAP_STATE_ESTABLISHED || major == GSS_S_CONTINUE_NEEDED);