s4-kdc: Remove obsolete kpasswdd heimdal implementation
authorJeremy Allison <jra@samba.org>
Mon, 12 Sep 2016 19:11:55 +0000 (12:11 -0700)
committerJeremy Allison <jra@samba.org>
Mon, 12 Sep 2016 22:19:26 +0000 (00:19 +0200)
Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
source4/kdc/kpasswd-heimdal.c [deleted file]

diff --git a/source4/kdc/kpasswd-heimdal.c b/source4/kdc/kpasswd-heimdal.c
deleted file mode 100644 (file)
index 49fc755..0000000
+++ /dev/null
@@ -1,551 +0,0 @@
-/*
-   Unix SMB/CIFS implementation.
-
-   kpasswd Server implementation
-
-   Copyright (C) Andrew Bartlett <abartlet@samba.org> 2005
-   Copyright (C) Andrew Tridgell       2005
-
-   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 3 of the License, or
-   (at your option) any later version.
-
-   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.
-
-   You should have received a copy of the GNU General Public License
-   along with this program.  If not, see <http://www.gnu.org/licenses/>.
-*/
-
-#include "includes.h"
-#include "smbd/service_task.h"
-#include "auth/gensec/gensec.h"
-#include "auth/credentials/credentials.h"
-#include "auth/auth.h"
-#include "dsdb/samdb/samdb.h"
-#include "../lib/util/util_ldb.h"
-#include "libcli/security/security.h"
-#include "param/param.h"
-#include "kdc/kdc-server.h"
-#include "kdc/kdc-glue.h"
-#include "dsdb/common/util.h"
-#include "kdc/kpasswd_glue.h"
-#include "kdc/kpasswd-helper.h"
-
-/* Return true if there is a valid error packet formed in the error_blob */
-static bool kpasswdd_make_unauth_error_reply(struct kdc_server *kdc,
-                                           TALLOC_CTX *mem_ctx,
-                                           uint16_t result_code,
-                                           const char *error_string,
-                                           DATA_BLOB *error_blob)
-{
-       bool ret;
-       int kret;
-       DATA_BLOB error_bytes;
-       krb5_data k5_error_bytes, k5_error_blob;
-       ret = kpasswd_make_error_reply(mem_ctx, result_code, error_string,
-                                      &error_bytes);
-       if (!ret) {
-               return false;
-       }
-       k5_error_bytes.data = error_bytes.data;
-       k5_error_bytes.length = error_bytes.length;
-       kret = smb_krb5_mk_error(kdc->smb_krb5_context->krb5_context,
-                                result_code,
-                                NULL,
-                                &k5_error_bytes,
-                                NULL,
-                                NULL,
-                                &k5_error_blob);
-       if (kret) {
-               return false;
-       }
-       *error_blob = data_blob_talloc(mem_ctx, k5_error_blob.data, k5_error_blob.length);
-       smb_krb5_free_data_contents(kdc->smb_krb5_context->krb5_context,
-                                   &k5_error_blob);
-       if (!error_blob->data) {
-               return false;
-       }
-       return true;
-}
-
-/*
-   A user password change
-
-   Return true if there is a valid error packet (or success) formed in
-   the error_blob
-*/
-static bool kpasswdd_change_password(struct kdc_server *kdc,
-                                    TALLOC_CTX *mem_ctx,
-                                    struct auth_session_info *session_info,
-                                    const DATA_BLOB *password,
-                                    DATA_BLOB *reply)
-{
-       NTSTATUS status;
-       NTSTATUS result = NT_STATUS_UNSUCCESSFUL;
-       enum samPwdChangeReason reject_reason;
-       struct samr_DomInfo1 *dominfo;
-       const char *error_string;
-
-       status = samdb_kpasswd_change_password(mem_ctx,
-                                              kdc->task->lp_ctx,
-                                              kdc->task->event_ctx,
-                                              kdc->samdb,
-                                              session_info,
-                                              password,
-                                              &reject_reason,
-                                              &dominfo,
-                                              &error_string,
-                                              &result);
-       if (!NT_STATUS_IS_OK(status)) {
-               return kpasswd_make_error_reply(mem_ctx,
-                                                KRB5_KPASSWD_ACCESSDENIED,
-                                                error_string,
-                                                reply);
-       }
-
-       return kpasswd_make_pwchange_reply(mem_ctx,
-                                          result,
-                                          reject_reason,
-                                          dominfo,
-                                          reply);
-}
-
-static bool kpasswd_process_request(struct kdc_server *kdc,
-                                   TALLOC_CTX *mem_ctx,
-                                   struct gensec_security *gensec_security,
-                                   uint16_t version,
-                                   DATA_BLOB *input,
-                                   DATA_BLOB *reply)
-{
-       struct auth_session_info *session_info;
-       size_t pw_len;
-
-       if (!NT_STATUS_IS_OK(gensec_session_info(gensec_security,
-                                                mem_ctx,
-                                                &session_info))) {
-               return kpasswd_make_error_reply(mem_ctx,
-                                               KRB5_KPASSWD_HARDERROR,
-                                               "gensec_session_info failed!",
-                                               reply);
-       }
-
-       switch (version) {
-       case KRB5_KPASSWD_VERS_CHANGEPW:
-       {
-               DATA_BLOB password;
-               if (!convert_string_talloc_handle(mem_ctx, lpcfg_iconv_handle(kdc->task->lp_ctx),
-                                              CH_UTF8, CH_UTF16,
-                                              (const char *)input->data,
-                                              input->length,
-                                              (void **)&password.data, &pw_len)) {
-                       return false;
-               }
-               password.length = pw_len;
-
-               return kpasswdd_change_password(kdc, mem_ctx, session_info,
-                                               &password, reply);
-       }
-       case KRB5_KPASSWD_VERS_SETPW:
-       {
-               NTSTATUS status;
-               enum samPwdChangeReason reject_reason = SAM_PWD_CHANGE_NO_ERROR;
-               struct samr_DomInfo1 *dominfo = NULL;
-               struct ldb_context *samdb;
-               krb5_context context = kdc->smb_krb5_context->krb5_context;
-
-               ChangePasswdDataMS chpw;
-               DATA_BLOB password;
-
-               krb5_principal principal;
-               char *set_password_on_princ;
-               struct ldb_dn *set_password_on_dn;
-               bool service_principal_name = false;
-
-               size_t len;
-               int ret;
-
-               ret = decode_ChangePasswdDataMS(input->data, input->length,
-                                               &chpw, &len);
-               if (ret) {
-                       return kpasswd_make_error_reply(mem_ctx,
-                                                       KRB5_KPASSWD_MALFORMED,
-                                                       "failed to decode password change structure",
-                                                       reply);
-               }
-
-               if (!convert_string_talloc_handle(mem_ctx, lpcfg_iconv_handle(kdc->task->lp_ctx),
-                                              CH_UTF8, CH_UTF16,
-                                              (const char *)chpw.newpasswd.data,
-                                              chpw.newpasswd.length,
-                                              (void **)&password.data, &pw_len)) {
-                       free_ChangePasswdDataMS(&chpw);
-                       return false;
-               }
-
-               password.length = pw_len;
-
-               if ((chpw.targname && !chpw.targrealm)
-                   || (!chpw.targname && chpw.targrealm)) {
-                       free_ChangePasswdDataMS(&chpw);
-                       return kpasswd_make_error_reply(mem_ctx,
-                                                       KRB5_KPASSWD_MALFORMED,
-                                                       "Realm and principal must be both present, or neither present",
-                                                       reply);
-               }
-               if (chpw.targname && chpw.targrealm) {
-                       ret = krb5_build_principal_ext(kdc->smb_krb5_context->krb5_context,
-                                                      &principal,
-                                                      strlen(*chpw.targrealm),
-                                                      *chpw.targrealm, 0);
-                       if (ret) {
-                               free_ChangePasswdDataMS(&chpw);
-                               return kpasswd_make_error_reply(mem_ctx,
-                                                               KRB5_KPASSWD_MALFORMED,
-                                                               "failed to get principal",
-                                                               reply);
-                       }
-                       if (copy_PrincipalName(chpw.targname, &principal->name)) {
-                               free_ChangePasswdDataMS(&chpw);
-                               krb5_free_principal(context, principal);
-                               return kpasswd_make_error_reply(mem_ctx,
-                                                               KRB5_KPASSWD_MALFORMED,
-                                                               "failed to extract principal to set",
-                                                               reply);
-                       }
-               } else {
-                       free_ChangePasswdDataMS(&chpw);
-                       return kpasswdd_change_password(kdc, mem_ctx, session_info,
-                                                       &password, reply);
-               }
-               free_ChangePasswdDataMS(&chpw);
-
-               if (principal->name.name_string.len >= 2) {
-                       service_principal_name = true;
-
-                       /* We use this, rather than 'no realm' flag,
-                        * as we don't want to accept a password
-                        * change on a principal from another realm */
-
-                       if (krb5_unparse_name_short(context, principal, &set_password_on_princ) != 0) {
-                               krb5_free_principal(context, principal);
-                               return kpasswd_make_error_reply(mem_ctx,
-                                                                KRB5_KPASSWD_MALFORMED,
-                                                                "krb5_unparse_name failed!",
-                                                                reply);
-                       }
-               } else {
-                       if (krb5_unparse_name(context, principal, &set_password_on_princ) != 0) {
-                               krb5_free_principal(context, principal);
-                               return kpasswd_make_error_reply(mem_ctx,
-                                                                KRB5_KPASSWD_MALFORMED,
-                                                                "krb5_unparse_name failed!",
-                                                                reply);
-                       }
-               }
-               krb5_free_principal(context, principal);
-
-               samdb = samdb_connect(mem_ctx, kdc->task->event_ctx, kdc->task->lp_ctx, session_info, 0);
-               if (!samdb) {
-                       free(set_password_on_princ);
-                       return kpasswd_make_error_reply(mem_ctx,
-                                                        KRB5_KPASSWD_HARDERROR,
-                                                        "Unable to open database!",
-                                                        reply);
-               }
-
-               DEBUG(3, ("%s\\%s (%s) is changing password of %s\n",
-                         session_info->info->domain_name,
-                         session_info->info->account_name,
-                         dom_sid_string(mem_ctx, &session_info->security_token->sids[PRIMARY_USER_SID_INDEX]),
-                         set_password_on_princ));
-               ret = ldb_transaction_start(samdb);
-               if (ret != LDB_SUCCESS) {
-                       free(set_password_on_princ);
-                       status = NT_STATUS_TRANSACTION_ABORTED;
-                       return kpasswd_make_pwchange_reply(mem_ctx,
-                                                          status,
-                                                          SAM_PWD_CHANGE_NO_ERROR,
-                                                          NULL,
-                                                          reply);
-               }
-
-               if (service_principal_name) {
-                       status = crack_service_principal_name(samdb, mem_ctx,
-                                                             set_password_on_princ,
-                                                             &set_password_on_dn, NULL);
-               } else {
-                       status = crack_user_principal_name(samdb, mem_ctx,
-                                                          set_password_on_princ,
-                                                          &set_password_on_dn, NULL);
-               }
-               free(set_password_on_princ);
-               if (!NT_STATUS_IS_OK(status)) {
-                       ldb_transaction_cancel(samdb);
-                       return kpasswd_make_pwchange_reply(mem_ctx,
-                                                          status,
-                                                          SAM_PWD_CHANGE_NO_ERROR,
-                                                          NULL,
-                                                          reply);
-               }
-
-               if (NT_STATUS_IS_OK(status)) {
-                       /* Admin password set */
-                       status = samdb_set_password(samdb, mem_ctx,
-                                                   set_password_on_dn, NULL,
-                                                   &password, NULL, NULL,
-                                                   NULL, NULL, /* this is not a user password change */
-                                                   &reject_reason, &dominfo);
-               }
-
-               if (NT_STATUS_IS_OK(status)) {
-                       ret = ldb_transaction_commit(samdb);
-                       if (ret != LDB_SUCCESS) {
-                               DEBUG(1,("Failed to commit transaction to set password on %s: %s\n",
-                                        ldb_dn_get_linearized(set_password_on_dn),
-                                        ldb_errstring(samdb)));
-                               status = NT_STATUS_TRANSACTION_ABORTED;
-                       }
-               } else {
-                       ldb_transaction_cancel(samdb);
-               }
-               return kpasswd_make_pwchange_reply(mem_ctx,
-                                                  status,
-                                                  reject_reason,
-                                                  dominfo,
-                                                  reply);
-       }
-       default:
-               return kpasswd_make_error_reply(mem_ctx,
-                                                KRB5_KPASSWD_BAD_VERSION,
-                                                talloc_asprintf(mem_ctx,
-                                                                "Protocol version %u not supported",
-                                                                version),
-                                                reply);
-       }
-}
-
-kdc_code kpasswdd_process(struct kdc_server *kdc,
-                         TALLOC_CTX *mem_ctx,
-                         DATA_BLOB *input,
-                         DATA_BLOB *reply,
-                         struct tsocket_address *peer_addr,
-                         struct tsocket_address *my_addr,
-                         int datagram_reply)
-{
-       bool ret;
-       const uint16_t header_len = 6;
-       uint16_t len;
-       uint16_t ap_req_len;
-       uint16_t krb_priv_len;
-       uint16_t version;
-       NTSTATUS nt_status;
-       DATA_BLOB ap_req, krb_priv_req;
-       DATA_BLOB krb_priv_rep = data_blob(NULL, 0);
-       DATA_BLOB ap_rep = data_blob(NULL, 0);
-       DATA_BLOB kpasswd_req, kpasswd_rep;
-       struct cli_credentials *server_credentials;
-       struct gensec_security *gensec_security;
-       TALLOC_CTX *tmp_ctx = talloc_new(mem_ctx);
-
-       char *keytab_name;
-
-       if (!tmp_ctx) {
-               return KDC_ERROR;
-       }
-
-       if (kdc->am_rodc) {
-               talloc_free(tmp_ctx);
-               return KDC_PROXY_REQUEST;
-       }
-
-       /* Be parinoid.  We need to ensure we don't just let the
-        * caller lead us into a buffer overflow */
-       if (input->length <= header_len) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       len = RSVAL(input->data, 0);
-       if (input->length != len) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* There are two different versions of this protocol so far,
-        * plus others in the standards pipe.  Fortunetly they all
-        * take a very similar framing */
-       version = RSVAL(input->data, 2);
-       ap_req_len = RSVAL(input->data, 4);
-       if ((ap_req_len >= len) || (ap_req_len + header_len) >= len) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       krb_priv_len = len - ap_req_len;
-       ap_req = data_blob_const(&input->data[header_len], ap_req_len);
-       krb_priv_req = data_blob_const(&input->data[header_len + ap_req_len], krb_priv_len);
-
-       server_credentials = cli_credentials_init(tmp_ctx);
-       if (!server_credentials) {
-               DEBUG(1, ("Failed to init server credentials\n"));
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* We want the credentials subsystem to use the krb5 context
-        * we already have, rather than a new context */
-       cli_credentials_set_krb5_context(server_credentials, kdc->smb_krb5_context);
-       cli_credentials_set_conf(server_credentials, kdc->task->lp_ctx);
-
-       keytab_name = talloc_asprintf(server_credentials, "HDB:samba4&%p", kdc->base_ctx);
-
-       cli_credentials_set_username(server_credentials, "kadmin/changepw", CRED_SPECIFIED);
-       ret = cli_credentials_set_keytab_name(server_credentials, kdc->task->lp_ctx, keytab_name, CRED_SPECIFIED);
-       if (ret != 0) {
-               ret = kpasswdd_make_unauth_error_reply(kdc, mem_ctx,
-                                                      KRB5_KPASSWD_HARDERROR,
-                                                      talloc_asprintf(mem_ctx,
-                                                                      "Failed to obtain server credentials for kadmin/changepw!"),
-                                                      &krb_priv_rep);
-               ap_rep.length = 0;
-               if (ret) {
-                       goto reply;
-               }
-               talloc_free(tmp_ctx);
-               return ret;
-       }
-
-       /* We don't strictly need to call this wrapper, and could call
-        * gensec_server_start directly, as we have no need for NTLM
-        * and we have a PAC, but this ensures that the wrapper can be
-        * safely extended for other helpful things in future */
-       nt_status = samba_server_gensec_start(tmp_ctx, kdc->task->event_ctx,
-                                             kdc->task->msg_ctx,
-                                             kdc->task->lp_ctx,
-                                             server_credentials,
-                                             "kpasswd",
-                                             &gensec_security);
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* The kerberos PRIV packets include these addresses.  MIT
-        * clients check that they are present */
-#if 0
-       /* Skip this part for now, it breaks with a NetAPP filer and
-        * in any case where the client address is behind NAT.  If
-        * older MIT clients need this, we might have to insert more
-        * complex code */
-
-       nt_status = gensec_set_remote_address(gensec_security, peer_addr);
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-#endif
-
-       nt_status = gensec_set_local_address(gensec_security, my_addr);
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* We want the GENSEC wrap calls to generate PRIV tokens */
-       gensec_want_feature(gensec_security, GENSEC_FEATURE_SEAL);
-
-       nt_status = gensec_start_mech_by_name(gensec_security, "krb5");
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* Accept the AP-REQ and generate teh AP-REP we need for the reply */
-       nt_status = gensec_update_ev(gensec_security, tmp_ctx, kdc->task->event_ctx, ap_req, &ap_rep);
-       if (!NT_STATUS_IS_OK(nt_status) && !NT_STATUS_EQUAL(nt_status, NT_STATUS_MORE_PROCESSING_REQUIRED)) {
-
-               ret = kpasswdd_make_unauth_error_reply(kdc, mem_ctx,
-                                                      KRB5_KPASSWD_HARDERROR,
-                                                      talloc_asprintf(mem_ctx,
-                                                                      "gensec_update failed: %s",
-                                                                      nt_errstr(nt_status)),
-                                                      &krb_priv_rep);
-               ap_rep.length = 0;
-               if (ret) {
-                       goto reply;
-               }
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* Extract the data from the KRB-PRIV half of the message */
-       nt_status = gensec_unwrap(gensec_security, tmp_ctx, &krb_priv_req, &kpasswd_req);
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               ret = kpasswdd_make_unauth_error_reply(kdc, mem_ctx,
-                                                      KRB5_KPASSWD_HARDERROR,
-                                                      talloc_asprintf(mem_ctx,
-                                                                      "gensec_unwrap failed: %s",
-                                                                      nt_errstr(nt_status)),
-                                                      &krb_priv_rep);
-               ap_rep.length = 0;
-               if (ret) {
-                       goto reply;
-               }
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* Figure out something to do with it (probably changing a password...) */
-       ret = kpasswd_process_request(kdc, tmp_ctx,
-                                     gensec_security,
-                                     version,
-                                     &kpasswd_req, &kpasswd_rep);
-       if (!ret) {
-               /* Argh! */
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       /* And wrap up the reply: This ensures that the error message
-        * or success can be verified by the client */
-       nt_status = gensec_wrap(gensec_security, tmp_ctx,
-                               &kpasswd_rep, &krb_priv_rep);
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               ret = kpasswdd_make_unauth_error_reply(kdc, mem_ctx,
-                                                      KRB5_KPASSWD_HARDERROR,
-                                                      talloc_asprintf(mem_ctx,
-                                                                      "gensec_wrap failed: %s",
-                                                                      nt_errstr(nt_status)),
-                                                      &krb_priv_rep);
-               ap_rep.length = 0;
-               if (ret) {
-                       goto reply;
-               }
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-reply:
-       *reply = data_blob_talloc(mem_ctx, NULL, krb_priv_rep.length + ap_rep.length + header_len);
-       if (!reply->data) {
-               talloc_free(tmp_ctx);
-               return KDC_ERROR;
-       }
-
-       RSSVAL(reply->data, 0, reply->length);
-       RSSVAL(reply->data, 2, 1); /* This is a version 1 reply, MS change/set or otherwise */
-       RSSVAL(reply->data, 4, ap_rep.length);
-       memcpy(reply->data + header_len,
-              ap_rep.data,
-              ap_rep.length);
-       memcpy(reply->data + header_len + ap_rep.length,
-              krb_priv_rep.data,
-              krb_priv_rep.length);
-
-       talloc_free(tmp_ctx);
-       return KDC_OK;
-}