Implemented pre-authentication certificate handling for IKEv1
authorMartin Willi <martin@revosec.ch>
Tue, 6 Dec 2011 11:14:48 +0000 (12:14 +0100)
committerMartin Willi <martin@revosec.ch>
Tue, 20 Mar 2012 16:31:13 +0000 (17:31 +0100)
src/libcharon/Makefile.am
src/libcharon/sa/ike_sa.c
src/libcharon/sa/task_manager_v1.c
src/libcharon/sa/tasks/ike_cert_pre_v1.c [new file with mode: 0755]
src/libcharon/sa/tasks/ike_cert_pre_v1.h [new file with mode: 0644]

index 4d3cc39..d36b5a5 100644 (file)
@@ -83,6 +83,7 @@ sa/tasks/child_rekey.c sa/tasks/child_rekey.h \
 sa/tasks/ike_auth.c sa/tasks/ike_auth.h \
 sa/tasks/ike_cert_pre.c sa/tasks/ike_cert_pre.h \
 sa/tasks/ike_cert_post.c sa/tasks/ike_cert_post.h \
+sa/tasks/ike_cert_pre_v1.c sa/tasks/ike_cert_pre_v1.h \
 sa/tasks/ike_config.c sa/tasks/ike_config.h \
 sa/tasks/ike_delete.c sa/tasks/ike_delete.h \
 sa/tasks/ike_dpd.c sa/tasks/ike_dpd.h \
index 5a21a88..96cbcf1 100644 (file)
@@ -35,6 +35,7 @@
 #include <sa/tasks/ike_auth_lifetime.h>
 #include <sa/tasks/ike_config.h>
 #include <sa/tasks/ike_cert_pre.h>
+#include <sa/tasks/ike_cert_pre_v1.h>
 #include <sa/tasks/ike_cert_post.h>
 #include <sa/tasks/ike_rekey.h>
 #include <sa/tasks/ike_reauth.h>
@@ -1122,6 +1123,8 @@ METHOD(ike_sa_t, initiate, status_t,
                {
                        task = (task_t*)ike_vendor_v1_create(&this->public, TRUE);
                        this->task_manager->queue_task(this->task_manager, task);
+                       task = (task_t*)ike_cert_pre_v1_create(&this->public, TRUE);
+                       this->task_manager->queue_task(this->task_manager, task);
                        task = (task_t*)main_mode_create(&this->public, TRUE);
                        this->task_manager->queue_task(this->task_manager, task);
                        task = (task_t*)ike_natd_v1_create(&this->public, TRUE);
index c08f70b..0843db5 100755 (executable)
@@ -24,7 +24,7 @@
 #include <sa/tasks/xauth_request.h>
 #include <sa/tasks/ike_natd_v1.h>
 #include <sa/tasks/ike_vendor_v1.h>
-#include <sa/tasks/ike_cert_pre.h>
+#include <sa/tasks/ike_cert_pre_v1.h>
 #include <sa/tasks/ike_cert_post.h>
 #include <processing/jobs/retransmit_job.h>
 #include <processing/jobs/delete_ike_sa_job.h>
@@ -266,6 +266,7 @@ METHOD(task_manager_t, initiate, status_t,
                {
                        case IKE_CREATED:
                                activate_task(this, TASK_VENDOR_V1);
+                               activate_task(this, TASK_IKE_CERT_PRE_V1);
                                if (activate_task(this, TASK_MAIN_MODE))
                                {
                                        exchange = ID_PROT;
@@ -501,14 +502,12 @@ static status_t process_request(private_task_manager_t *this,
                        case ID_PROT:
                                task = (task_t *)ike_vendor_v1_create(this->ike_sa, FALSE);
                                this->passive_tasks->insert_last(this->passive_tasks, task);
-                               task = (task_t*)ike_cert_pre_create(this->ike_sa, FALSE);
+                               task = (task_t*)ike_cert_pre_v1_create(this->ike_sa, FALSE);
                                this->passive_tasks->insert_last(this->passive_tasks, task);
                                task = (task_t *)main_mode_create(this->ike_sa, FALSE);
                                this->passive_tasks->insert_last(this->passive_tasks, task);
                                task = (task_t *)ike_natd_v1_create(this->ike_sa, FALSE);
                                this->passive_tasks->insert_last(this->passive_tasks, task);
-                               task = (task_t*)ike_cert_post_create(this->ike_sa, FALSE);
-                               this->passive_tasks->insert_last(this->passive_tasks, task);
                                break;
                        case AGGRESSIVE:
                                /* TODO-IKEv1: agressive mode */
diff --git a/src/libcharon/sa/tasks/ike_cert_pre_v1.c b/src/libcharon/sa/tasks/ike_cert_pre_v1.c
new file mode 100755 (executable)
index 0000000..fab8d5b
--- /dev/null
@@ -0,0 +1,517 @@
+/*
+ * Copyright (C) 2011 Martin Willi
+ * Copyright (C) 2011 revosec AG
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.  See <http://www.fsf.org/copyleft/gpl.txt>.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * for more details.
+ */
+
+#include "ike_cert_pre_v1.h"
+
+#include <daemon.h>
+#include <sa/ike_sa.h>
+#include <encoding/payloads/cert_payload.h>
+#include <encoding/payloads/sa_payload.h>
+#include <encoding/payloads/certreq_payload.h>
+#include <credentials/certificates/x509.h>
+
+
+typedef struct private_ike_cert_pre_v1_t private_ike_cert_pre_v1_t;
+
+/**
+ * Private members of a ike_cert_pre_v1_t task.
+ */
+struct private_ike_cert_pre_v1_t {
+
+       /**
+        * Public methods and task_t interface.
+        */
+       ike_cert_pre_v1_t public;
+
+       /**
+        * Assigned IKE_SA.
+        */
+       ike_sa_t *ike_sa;
+
+       /**
+        * Are we the initiator?
+        */
+       bool initiator;
+
+       /** next message we expect */
+       enum {
+               CR_SA,
+               CR_KE,
+               CR_AUTH,
+       } state;
+};
+
+/**
+ * Find the CA certificate for a given certreq payload
+ */
+static certificate_t* find_certificate(private_ike_cert_pre_v1_t *this,
+                                                                          certreq_payload_t *certreq)
+{
+       identification_t *id;
+       certificate_t *cert;
+
+       if (certreq->get_cert_type(certreq) != CERT_X509)
+       {
+               DBG1(DBG_IKE, "%N CERTREQ not supported - ignored",
+                        certificate_type_names, certreq->get_cert_type(certreq));
+               return NULL;
+       }
+       id = certreq->get_dn(certreq);
+       if (!id)
+       {
+               DBG1(DBG_IKE, "ignoring certificate request without data",
+                        certificate_type_names, certreq->get_cert_type(certreq));
+               return NULL;
+       }
+       cert = lib->credmgr->get_cert(lib->credmgr, CERT_X509, KEY_ANY, id, TRUE);
+       if (cert)
+       {
+               DBG1(DBG_IKE, "received cert request for '%Y'",
+                        cert->get_subject(cert));
+       }
+       else
+       {
+               DBG1(DBG_IKE, "received cert request for unknown ca '%Y'", id);
+       }
+       id->destroy(id);
+
+       return cert;
+}
+
+/**
+ * read certificate requests
+ */
+static void process_certreqs(private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       enumerator_t *enumerator;
+       payload_t *payload;
+       auth_cfg_t *auth;
+
+       auth = this->ike_sa->get_auth_cfg(this->ike_sa, TRUE);
+
+       enumerator = message->create_payload_enumerator(message);
+       while (enumerator->enumerate(enumerator, &payload))
+       {
+               switch (payload->get_type(payload))
+               {
+                       case CERTIFICATE_REQUEST_V1:
+                       {
+                               certificate_t *cert;
+
+                               this->ike_sa->set_condition(this->ike_sa,
+                                                                                       COND_CERTREQ_SEEN, TRUE);
+                               cert = find_certificate(this, (certreq_payload_t*)payload);
+                               if (cert)
+                               {
+                                       auth->add(auth, AUTH_RULE_CA_CERT, cert);
+                               }
+                               break;
+                       }
+                       default:
+                               break;
+               }
+       }
+       enumerator->destroy(enumerator);
+}
+
+/**
+ * Import receuved certificates
+ */
+static void process_certs(private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       enumerator_t *enumerator;
+       payload_t *payload;
+       auth_cfg_t *auth;
+       bool first = TRUE;
+
+       auth = this->ike_sa->get_auth_cfg(this->ike_sa, FALSE);
+
+       enumerator = message->create_payload_enumerator(message);
+       while (enumerator->enumerate(enumerator, &payload))
+       {
+               if (payload->get_type(payload) == CERTIFICATE_V1)
+               {
+                       cert_payload_t *cert_payload;
+                       cert_encoding_t encoding;
+                       certificate_t *cert;
+
+                       cert_payload = (cert_payload_t*)payload;
+                       encoding = cert_payload->get_cert_encoding(cert_payload);
+
+                       switch (encoding)
+                       {
+                               case ENC_X509_SIGNATURE:
+                               {
+                                       cert = cert_payload->get_cert(cert_payload);
+                                       if (cert)
+                                       {
+                                               if (first)
+                                               {       /* the first is an end entity certificate */
+                                                       DBG1(DBG_IKE, "received end entity cert \"%Y\"",
+                                                                cert->get_subject(cert));
+                                                       auth->add(auth, AUTH_HELPER_SUBJECT_CERT, cert);
+                                                       first = FALSE;
+                                               }
+                                               else
+                                               {
+                                                       DBG1(DBG_IKE, "received issuer cert \"%Y\"",
+                                                                cert->get_subject(cert));
+                                                       auth->add(auth, AUTH_HELPER_IM_CERT, cert);
+                                               }
+                                       }
+                                       break;
+                               }
+                               case ENC_CRL:
+                                       cert = cert_payload->get_cert(cert_payload);
+                                       if (cert)
+                                       {
+                                               DBG1(DBG_IKE, "received CRL \"%Y\"",
+                                                        cert->get_subject(cert));
+                                               auth->add(auth, AUTH_HELPER_REVOCATION_CERT, cert);
+                                       }
+                                       break;
+                               case ENC_PKCS7_WRAPPED_X509:
+                               case ENC_PGP:
+                               case ENC_DNS_SIGNED_KEY:
+                               case ENC_KERBEROS_TOKEN:
+                               case ENC_ARL:
+                               case ENC_SPKI:
+                               case ENC_X509_ATTRIBUTE:
+                               case ENC_RAW_RSA_KEY:
+                               case ENC_X509_HASH_AND_URL_BUNDLE:
+                               case ENC_OCSP_CONTENT:
+                               default:
+                                       DBG1(DBG_ENC, "certificate encoding %N not supported",
+                                                cert_encoding_names, encoding);
+                       }
+               }
+       }
+       enumerator->destroy(enumerator);
+}
+
+/**
+ * Add the subject of a CA certificate a message
+ */
+static void add_certreq(private_ike_cert_pre_v1_t *this, message_t *message,
+                                               certificate_t *cert)
+{
+       if (cert->get_type(cert) == CERT_X509)
+       {
+               x509_t *x509 = (x509_t*)cert;
+
+               if (x509->get_flags(x509) & X509_CA)
+               {
+                       DBG1(DBG_IKE, "sending cert request for \"%Y\"",
+                                cert->get_subject(cert));
+                       message->add_payload(message, (payload_t*)
+                                                       certreq_payload_create_dn(cert->get_subject(cert)));
+               }
+       }
+}
+
+/**
+ * Add auth_cfg's CA certificates to the certificate request
+ */
+static void add_certreqs(private_ike_cert_pre_v1_t *this,
+                                                auth_cfg_t *auth, message_t *message)
+{
+       enumerator_t *enumerator;
+       auth_rule_t type;
+       void *value;
+
+       enumerator = auth->create_enumerator(auth);
+       while (enumerator->enumerate(enumerator, &type, &value))
+       {
+               switch (type)
+               {
+                       case AUTH_RULE_CA_CERT:
+                               add_certreq(this, message, (certificate_t*)value);
+                               break;
+                       default:
+                               break;
+               }
+       }
+       enumerator->destroy(enumerator);
+}
+
+/**
+ * Build certificate requests
+ */
+static void build_certreqs(private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       enumerator_t *enumerator;
+       ike_cfg_t *ike_cfg;
+       peer_cfg_t *peer_cfg;
+       certificate_t *cert;
+       auth_cfg_t *auth;
+
+       ike_cfg = this->ike_sa->get_ike_cfg(this->ike_sa);
+       if (!ike_cfg->send_certreq(ike_cfg))
+       {
+               return;
+       }
+       /* check if we require a specific CA for that peer */
+       peer_cfg = this->ike_sa->get_peer_cfg(this->ike_sa);
+       if (peer_cfg)
+       {
+               enumerator = peer_cfg->create_auth_cfg_enumerator(peer_cfg, FALSE);
+               if (enumerator->enumerate(enumerator, &auth))
+               {
+                       add_certreqs(this, auth, message);
+               }
+               enumerator->destroy(enumerator);
+       }
+       if (!message->get_payload(message, CERTIFICATE_REQUEST_V1))
+       {
+               /* otherwise add all trusted CA certificates */
+               enumerator = lib->credmgr->create_cert_enumerator(lib->credmgr,
+                                                                                               CERT_ANY, KEY_ANY, NULL, TRUE);
+               while (enumerator->enumerate(enumerator, &cert))
+               {
+                       add_certreq(this, message, cert);
+               }
+               enumerator->destroy(enumerator);
+       }
+}
+
+/**
+ * Check if we actually use certificates for authentication
+ */
+static bool use_certs(private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       enumerator_t *enumerator;
+       payload_t *payload;
+       bool use = FALSE;
+
+       enumerator = message->create_payload_enumerator(message);
+       while (enumerator->enumerate(enumerator, &payload))
+       {
+               if (payload->get_type(payload) == SECURITY_ASSOCIATION_V1)
+               {
+                       sa_payload_t *sa_payload = (sa_payload_t*)payload;
+
+                       switch (sa_payload->get_auth_method(sa_payload))
+                       {
+                               case AUTH_RSA:
+                               case AUTH_XAUTH_INIT_RSA:
+                               case AUTH_XAUTH_RESP_RSA:
+                                       use = TRUE;
+                                       break;
+                               default:
+                                       break;
+                       }
+                       break;
+               }
+       }
+       enumerator->destroy(enumerator);
+
+       return use;
+}
+
+METHOD(task_t, build_i, status_t,
+       private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       switch (message->get_exchange_type(message))
+       {
+               case ID_PROT:
+                       if (this->state == CR_AUTH)
+                       {
+                               build_certreqs(this, message);
+                       }
+                       return NEED_MORE;
+               case AGGRESSIVE:
+                       if (this->state == CR_SA)
+                       {
+                               build_certreqs(this, message);
+                       }
+                       return NEED_MORE;
+               default:
+                       return FAILED;
+       }
+}
+
+METHOD(task_t, process_r, status_t,
+       private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       switch (message->get_exchange_type(message))
+       {
+               case ID_PROT:
+               {
+                       switch (this->state)
+                       {
+                               case CR_SA:
+                                       if (!use_certs(this, message))
+                                       {
+                                               return SUCCESS;
+                                       }
+                                       return NEED_MORE;
+                               case CR_KE:
+                                       process_certreqs(this, message);
+                                       return NEED_MORE;
+                               case CR_AUTH:
+                                       process_certreqs(this, message);
+                                       process_certs(this, message);
+                                       return SUCCESS;
+                       }
+               }
+               case AGGRESSIVE:
+               {
+                       switch (this->state)
+                       {
+                               case CR_SA:
+                                       if (!use_certs(this, message))
+                                       {
+                                               return SUCCESS;
+                                       }
+                                       process_certreqs(this, message);
+                                       return NEED_MORE;
+                               case CR_AUTH:
+                                       process_certs(this, message);
+                                       return SUCCESS;
+                               default:
+                                       return FAILED;
+                       }
+               }
+               default:
+                       return FAILED;
+       }
+}
+
+METHOD(task_t, build_r, status_t,
+       private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       switch (message->get_exchange_type(message))
+       {
+               case ID_PROT:
+                       switch (this->state)
+                       {
+                               case CR_SA:
+                                       this->state = CR_KE;
+                                       return NEED_MORE;
+                               case CR_KE:
+                                       build_certreqs(this, message);
+                                       this->state = CR_AUTH;
+                                       return NEED_MORE;
+                               case CR_AUTH:
+                                       return NEED_MORE;
+                       }
+               case AGGRESSIVE:
+                       switch (this->state)
+                       {
+                               case CR_SA:
+                                       this->state = CR_AUTH;
+                                       return NEED_MORE;
+                               case CR_AUTH:
+                                       return SUCCESS;
+                               default:
+                                       return FAILED;
+                       }
+               default:
+                       return FAILED;
+       }
+}
+
+METHOD(task_t, process_i, status_t,
+       private_ike_cert_pre_v1_t *this, message_t *message)
+{
+       switch (message->get_exchange_type(message))
+       {
+               case ID_PROT:
+               {
+                       switch (this->state)
+                       {
+                               case CR_SA:
+                                       if (!use_certs(this, message))
+                                       {
+                                               return SUCCESS;
+                                       }
+                                       this->state = CR_KE;
+                                       return NEED_MORE;
+                               case CR_KE:
+                                       process_certreqs(this, message);
+                                       this->state = CR_AUTH;
+                                       return NEED_MORE;
+                               case CR_AUTH:
+                                       process_certs(this, message);
+                                       return SUCCESS;
+                               default:
+                                       return FAILED;
+                       }
+                       break;
+               }
+               case AGGRESSIVE:
+               {
+                       if (!use_certs(this, message))
+                       {
+                               return SUCCESS;
+                       }
+                       process_certreqs(this, message);
+                       process_certs(this, message);
+                       return SUCCESS;
+               }
+               default:
+                       return FAILED;
+       }
+}
+
+METHOD(task_t, get_type, task_type_t,
+       private_ike_cert_pre_v1_t *this)
+{
+       return TASK_IKE_CERT_PRE_V1;
+}
+
+METHOD(task_t, migrate, void,
+       private_ike_cert_pre_v1_t *this, ike_sa_t *ike_sa)
+{
+       this->ike_sa = ike_sa;
+}
+
+METHOD(task_t, destroy, void,
+       private_ike_cert_pre_v1_t *this)
+{
+       free(this);
+}
+
+/*
+ * Described in header.
+ */
+ike_cert_pre_v1_t *ike_cert_pre_v1_create(ike_sa_t *ike_sa, bool initiator)
+{
+       private_ike_cert_pre_v1_t *this;
+
+       INIT(this,
+               .public = {
+                       .task = {
+                               .get_type = _get_type,
+                               .migrate = _migrate,
+                               .destroy = _destroy,
+                       },
+               },
+               .ike_sa = ike_sa,
+               .initiator = initiator,
+               .state = CR_SA,
+       );
+       if (initiator)
+       {
+               this->public.task.build = _build_i;
+               this->public.task.process = _process_i;
+       }
+       else
+       {
+               this->public.task.build = _build_r;
+               this->public.task.process = _process_r;
+       }
+       return &this->public;
+}
diff --git a/src/libcharon/sa/tasks/ike_cert_pre_v1.h b/src/libcharon/sa/tasks/ike_cert_pre_v1.h
new file mode 100644 (file)
index 0000000..38042cc
--- /dev/null
@@ -0,0 +1,53 @@
+/*
+ * Copyright (C) 2011 Martin Willi
+ * Copyright (C) 2011 revosec AG
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.  See <http://www.fsf.org/copyleft/gpl.txt>.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * for more details.
+ */
+
+/**
+ * @defgroup ike_cert_pre_v1 ike_cert_pre_v1
+ * @{ @ingroup tasks
+ */
+
+#ifndef IKE_CERT_PRE_V1_H_
+#define IKE_CERT_PRE_V1_H_
+
+typedef struct ike_cert_pre_v1_t ike_cert_pre_v1_t;
+
+#include <library.h>
+#include <sa/ike_sa.h>
+#include <sa/tasks/task.h>
+
+/**
+ * IKE_CERT_PRE_V1 task, IKEv1 certificate processing before authentication.
+ */
+struct ike_cert_pre_v1_t {
+
+       /**
+        * Implements the task_t interface
+        */
+       task_t task;
+};
+
+/**
+ * Create a new IKE_CERT_PRE_V1 task.
+ *
+ * The initiator parameter means the original initiator, not the initiator
+ * of the certificate request.
+ *
+ * @param ike_sa               IKE_SA this task works for
+ * @param initiator            TRUE if task is the original initiator
+ * @return                             ike_cert_pre_v1 task to handle by the task_manager
+ */
+ike_cert_pre_v1_t *ike_cert_pre_v1_create(ike_sa_t *ike_sa, bool initiator);
+
+#endif /** IIKE_CERT_PRE_V1_H_ @}*/