From b3cf15e530fc41c4cf78e7adf3507b68436f2544 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Mon, 12 Sep 2016 12:11:55 -0700 Subject: [PATCH] s4-kdc: Remove obsolete kpasswdd heimdal implementation Signed-off-by: Jeremy Allison Reviewed-by: Andreas Schneider --- source4/kdc/kpasswd-heimdal.c | 551 ---------------------------------- 1 file changed, 551 deletions(-) delete mode 100644 source4/kdc/kpasswd-heimdal.c diff --git a/source4/kdc/kpasswd-heimdal.c b/source4/kdc/kpasswd-heimdal.c deleted file mode 100644 index 49fc755dad39..000000000000 --- a/source4/kdc/kpasswd-heimdal.c +++ /dev/null @@ -1,551 +0,0 @@ -/* - Unix SMB/CIFS implementation. - - kpasswd Server implementation - - Copyright (C) Andrew Bartlett 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 . -*/ - -#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; -} -- 2.34.1