Merge branch 'v4-0-test' of ssh://git.samba.org/data/git/samba into registry
[ira/wip.git] / source4 / kdc / kpasswdd.c
index 4dd0eda44429193ce7c4c52c13c9fccc2c5c0071..1d49a8a4bd23d726fe56754cd6fd6cd3fbfe63b1 100644 (file)
@@ -8,7 +8,7 @@
 
    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
+   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,
@@ -17,8 +17,7 @@
    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, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "includes.h"
 #include "lib/socket/socket.h"
 #include "kdc/kdc.h"
 #include "system/network.h"
-#include "dlinklist.h"
+#include "lib/util/dlinklist.h"
 #include "lib/ldb/include/ldb.h"
+#include "heimdal/lib/krb5/krb5_locl.h"
 #include "heimdal/lib/krb5/krb5-private.h"
+#include "auth/gensec/gensec.h"
+#include "auth/credentials/credentials.h"
+#include "auth/credentials/credentials_krb5.h"
 #include "auth/auth.h"
+#include "dsdb/samdb/samdb.h"
+#include "rpc_server/dcerpc_server.h"
+#include "rpc_server/samr/proto.h"
+#include "libcli/security/security.h"
+#include "param/param.h"
 
 /* hold information about one kdc socket */
 struct kpasswd_socket {
@@ -43,7 +51,7 @@ struct kpasswd_socket {
 };
 
 /* Return true if there is a valid error packet formed in the error_blob */
-static BOOL kpasswdd_make_error_reply(struct kdc_server *kdc, 
+static bool kpasswdd_make_error_reply(struct kdc_server *kdc, 
                                     TALLOC_CTX *mem_ctx, 
                                     uint16_t result_code, 
                                     const char *error_string, 
@@ -54,35 +62,35 @@ static BOOL kpasswdd_make_error_reply(struct kdc_server *kdc,
        
        DEBUG(result_code ? 3 : 10, ("kpasswdd: %s\n", error_string));
 
-       len = push_utf8_talloc(mem_ctx, &error_string_utf8, error_string);
+       len = push_utf8_talloc(mem_ctx, lp_iconv_convenience(kdc->task->lp_ctx), &error_string_utf8, error_string);
        if (len == -1) {
-               return False;
+               return false;
        }
 
        *error_blob = data_blob_talloc(mem_ctx, NULL, 2 + len + 1);
        if (!error_blob->data) {
-               return False;
+               return false;
        }
        RSSVAL(error_blob->data, 0, result_code);
        memcpy(error_blob->data + 2, error_string_utf8, len + 1);
-       return True;
+       return true;
 }
 
 /* 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, 
+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;
+       bool ret;
        int kret;
        DATA_BLOB error_bytes;
        krb5_data k5_error_bytes, k5_error_blob;
        ret = kpasswdd_make_error_reply(kdc, mem_ctx, result_code, error_string, 
                                       &error_bytes);
        if (!ret) {
-               return False;
+               return false;
        }
        k5_error_bytes.data = error_bytes.data;
        k5_error_bytes.length = error_bytes.length;
@@ -90,17 +98,17 @@ static BOOL kpasswdd_make_unauth_error_reply(struct kdc_server *kdc,
                             result_code, NULL, &k5_error_bytes, 
                             NULL, NULL, NULL, NULL, &k5_error_blob);
        if (kret) {
-               return False;
+               return false;
        }
        *error_blob = data_blob_talloc(mem_ctx, k5_error_blob.data, k5_error_blob.length);
        krb5_data_free(&k5_error_blob);
        if (!error_blob->data) {
-               return False;
+               return false;
        }
-       return True;
+       return true;
 }
 
-static BOOL kpasswd_make_pwchange_reply(struct kdc_server *kdc, 
+static bool kpasswd_make_pwchange_reply(struct kdc_server *kdc, 
                                        TALLOC_CTX *mem_ctx, 
                                        NTSTATUS status, 
                                        enum samr_RejectReason reject_reason,
@@ -119,7 +127,7 @@ static BOOL kpasswd_make_pwchange_reply(struct kdc_server *kdc,
                                                "Not permitted to change password",
                                                error_blob);
        }
-       if (NT_STATUS_EQUAL(status, NT_STATUS_PASSWORD_RESTRICTION)) {
+       if (dominfo && NT_STATUS_EQUAL(status, NT_STATUS_PASSWORD_RESTRICTION)) {
                const char *reject_string;
                switch (reject_reason) {
                case SAMR_REJECT_TOO_SHORT:
@@ -129,6 +137,9 @@ static BOOL kpasswd_make_pwchange_reply(struct kdc_server *kdc,
                case SAMR_REJECT_COMPLEXITY:
                        reject_string = "Password does not meet complexity requirements";
                        break;
+               case SAMR_REJECT_IN_HISTORY:
+                       reject_string = "Password is already in password history";
+                       break;
                case SAMR_REJECT_OTHER:
                default:
                        reject_string = talloc_asprintf(mem_ctx, "Password must be at least %d characters long, and cannot match any of your %d previous passwords",
@@ -158,7 +169,7 @@ static BOOL kpasswd_make_pwchange_reply(struct kdc_server *kdc,
    Return true if there is a valid error packet (or sucess) formed in
    the error_blob
 */
-static BOOL kpasswdd_change_password(struct kdc_server *kdc,
+static bool kpasswdd_change_password(struct kdc_server *kdc,
                                     TALLOC_CTX *mem_ctx, 
                                     struct auth_session_info *session_info,
                                     const char *password,
@@ -169,7 +180,7 @@ static BOOL kpasswdd_change_password(struct kdc_server *kdc,
        struct samr_DomInfo1 *dominfo;
        struct ldb_context *samdb;
 
-       samdb = samdb_connect(mem_ctx, system_session(mem_ctx));
+       samdb = samdb_connect(mem_ctx, kdc->task->event_ctx, kdc->task->lp_ctx, system_session(mem_ctx, kdc->task->lp_ctx));
        if (!samdb) {
                return kpasswdd_make_error_reply(kdc, mem_ctx, 
                                                KRB5_KPASSWD_HARDERROR,
@@ -177,14 +188,16 @@ static BOOL kpasswdd_change_password(struct kdc_server *kdc,
                                                reply);
        }
        
-       DEBUG(3, ("Changing password of %s\n", dom_sid_string(mem_ctx, session_info->security_token->user_sid)));
+       DEBUG(3, ("Changing password of %s\\%s (%s)\n", 
+                 session_info->server_info->domain_name,
+                 session_info->server_info->account_name,
+                 dom_sid_string(mem_ctx, session_info->security_token->user_sid)));
 
        /* User password change */
        status = samdb_set_password_sid(samdb, mem_ctx, 
                                        session_info->security_token->user_sid,
                                        password, NULL, NULL, 
-                                       True, /* this is a user password change */
-                                       True, /* run restriction tests */
+                                       true, /* this is a user password change */
                                        &reject_reason,
                                        &dominfo);
        return kpasswd_make_pwchange_reply(kdc, mem_ctx, 
@@ -195,25 +208,14 @@ static BOOL kpasswdd_change_password(struct kdc_server *kdc,
 
 }
 
-static BOOL kpasswd_process_request(struct kdc_server *kdc,
+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)
 {
-       NTSTATUS status;
-       enum samr_RejectReason reject_reason;
-       struct samr_DomInfo1 *dominfo;
-       struct ldb_context *samdb;
        struct auth_session_info *session_info;
-       struct ldb_message *msg = ldb_msg_new(gensec_security);
-       krb5_context context = kdc->smb_krb5_context->krb5_context;
-       int ret;
-       if (!msg) {
-               return False;
-       }
-
        if (!NT_STATUS_IS_OK(gensec_session_info(gensec_security, 
                                                 &session_info))) {
                return kpasswdd_make_error_reply(kdc, mem_ctx, 
@@ -227,7 +229,7 @@ static BOOL kpasswd_process_request(struct kdc_server *kdc,
        {
                char *password = talloc_strndup(mem_ctx, (const char *)input->data, input->length);
                if (!password) {
-                       return False;
+                       return false;
                }
                return kpasswdd_change_password(kdc, mem_ctx, session_info, 
                                                password, reply);
@@ -235,14 +237,27 @@ static BOOL kpasswd_process_request(struct kdc_server *kdc,
        }
        case KRB5_KPASSWD_VERS_SETPW:
        {
-               size_t len;
+               NTSTATUS status;
+               enum samr_RejectReason reject_reason = SAMR_REJECT_OTHER;
+               struct samr_DomInfo1 *dominfo = NULL;
+               struct ldb_context *samdb;
+               struct ldb_message *msg;
+               krb5_context context = kdc->smb_krb5_context->krb5_context;
+
                ChangePasswdDataMS chpw;
                char *password;
+
                krb5_principal principal;
                char *set_password_on_princ;
                struct ldb_dn *set_password_on_dn;
 
-               samdb = samdb_connect(gensec_security, session_info);
+               size_t len;
+               int ret;
+
+               msg = ldb_msg_new(mem_ctx);
+               if (!msg) {
+                       return false;
+               }
 
                ret = decode_ChangePasswdDataMS(input->data, input->length,
                                                &chpw, &len);
@@ -253,11 +268,12 @@ static BOOL kpasswd_process_request(struct kdc_server *kdc,
                                                        reply);
                }
                
-               password = talloc_strndup(mem_ctx, chpw.newpasswd.data, 
+               password = talloc_strndup(mem_ctx, 
+                                         (const char *)chpw.newpasswd.data, 
                                          chpw.newpasswd.length);
                if (!password) {
                        free_ChangePasswdDataMS(&chpw);
-                       return False;
+                       return false;
                }
                if ((chpw.targname && !chpw.targrealm) 
                    || (!chpw.targname && chpw.targrealm)) {
@@ -267,7 +283,8 @@ static BOOL kpasswd_process_request(struct kdc_server *kdc,
                                                        reply);
                }
                if (chpw.targname && chpw.targrealm) {
-                       if (_krb5_principalname2krb5_principal(&principal, *chpw.targname, 
+                       if (_krb5_principalname2krb5_principal(kdc->smb_krb5_context->krb5_context,
+                                                              &principal, *chpw.targname, 
                                                               *chpw.targrealm) != 0) {
                                free_ChangePasswdDataMS(&chpw);
                                return kpasswdd_make_error_reply(kdc, mem_ctx, 
@@ -293,26 +310,83 @@ static BOOL kpasswd_process_request(struct kdc_server *kdc,
                
                krb5_free_principal(context, principal);
                
+               samdb = samdb_connect(mem_ctx, kdc->task->event_ctx, kdc->task->lp_ctx, session_info);
+               if (!samdb) {
+                       return kpasswdd_make_error_reply(kdc, mem_ctx, 
+                                                        KRB5_KPASSWD_HARDERROR,
+                                                        "Unable to open database!",
+                                                        reply);
+               }
+
+               DEBUG(3, ("%s\\%s (%s) is changing password of %s\n", 
+                         session_info->server_info->domain_name,
+                         session_info->server_info->account_name,
+                         dom_sid_string(mem_ctx, session_info->security_token->user_sid), 
+                         set_password_on_princ));
+               ret = ldb_transaction_start(samdb);
+               if (ret) {
+                       status = NT_STATUS_TRANSACTION_ABORTED;
+                       return kpasswd_make_pwchange_reply(kdc, mem_ctx, 
+                                                          status,
+                                                          SAMR_REJECT_OTHER, 
+                                                          NULL, 
+                                                          reply);
+               }
+
                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(kdc, mem_ctx, 
                                                           status,
-                                                          reject_reason
-                                                          dominfo
+                                                          SAMR_REJECT_OTHER
+                                                          NULL
                                                           reply);
                }
 
-               /* Admin password set */
-               status = samdb_set_password(samdb, mem_ctx,
-                                           set_password_on_dn, NULL,
-                                           msg, password, NULL, NULL, 
-                                           False, /* this is not a user password change */
-                                           True, /* run restriction tests */
-                                           &reject_reason, &dominfo);
+               msg = ldb_msg_new(mem_ctx);
+               if (msg == NULL) {
+                       ldb_transaction_cancel(samdb);
+                       status = NT_STATUS_NO_MEMORY;
+               } else {
+                       msg->dn = ldb_dn_copy(msg, set_password_on_dn);
+                       if (!msg->dn) {
+                               status = NT_STATUS_NO_MEMORY;
+                       }
+               }
+
+               if (NT_STATUS_IS_OK(status)) {
+                       /* Admin password set */
+                       status = samdb_set_password(samdb, mem_ctx,
+                                                   set_password_on_dn, NULL,
+                                                   msg, password, NULL, NULL, 
+                                                   false, /* this is not a user password change */
+                                                   &reject_reason, &dominfo);
+               }
 
+               if (NT_STATUS_IS_OK(status)) {
+                       /* modify the samdb record */
+                       ret = samdb_replace(samdb, mem_ctx, msg);
+                       if (ret != 0) {
+                               DEBUG(2,("Failed to modify record to set password on %s: %s\n",
+                                        ldb_dn_get_linearized(msg->dn),
+                                        ldb_errstring(samdb)));
+                               status = NT_STATUS_ACCESS_DENIED;
+                       }
+               }
+               if (NT_STATUS_IS_OK(status)) {
+                       ret = ldb_transaction_commit(samdb);
+                       if (ret != 0) {
+                               DEBUG(1,("Failed to commit transaction to set password on %s: %s\n",
+                                        ldb_dn_get_linearized(msg->dn),
+                                        ldb_errstring(samdb)));
+                               status = NT_STATUS_TRANSACTION_ABORTED;
+                       }
+               } else {
+                       ldb_transaction_cancel(samdb);
+               }
                return kpasswd_make_pwchange_reply(kdc, mem_ctx, 
                                                   status,
                                                   reject_reason, 
@@ -321,76 +395,86 @@ static BOOL kpasswd_process_request(struct kdc_server *kdc,
        }
        default:
                return kpasswdd_make_error_reply(kdc, mem_ctx, 
-                                               KRB5_KPASSWD_BAD_VERSION,
-                                               talloc_asprintf(mem_ctx, 
-                                                               "Protocol version %u not supported", 
-                                                               version),
-                                               reply);
+                                                KRB5_KPASSWD_BAD_VERSION,
+                                                talloc_asprintf(mem_ctx, 
+                                                                "Protocol version %u not supported", 
+                                                                version),
+                                                reply);
        }
-       return True;
+       return true;
 }
 
-BOOL kpasswdd_process(struct kdc_server *kdc,
+bool kpasswdd_process(struct kdc_server *kdc,
                      TALLOC_CTX *mem_ctx, 
                      DATA_BLOB *input, 
                      DATA_BLOB *reply,
-                     const char *from,
-                     int src_port)
+                     struct socket_address *peer_addr,
+                     struct socket_address *my_addr,
+                     int datagram_reply)
 {
-       BOOL ret;
+       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, krb_priv_rep, ap_rep;
+       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);
        
        if (!tmp_ctx) {
-               return False;
+               return false;
        }
 
+       /* 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 False;
+               return false;
        }
 
        len = RSVAL(input->data, 0);
        if (input->length != len) {
                talloc_free(tmp_ctx);
-               return False;
+               return false;
        }
 
+       /* 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 False;
+               return false;
        }
        
        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);
        
-       nt_status = gensec_server_start(tmp_ctx, &gensec_security, kdc->task->event_ctx);
+       nt_status = gensec_server_start(tmp_ctx, kdc->task->event_ctx, kdc->task->lp_ctx, kdc->task->msg_ctx, &gensec_security);
        if (!NT_STATUS_IS_OK(nt_status)) {
                talloc_free(tmp_ctx);
-               return False;
+               return false;
        }
 
-       server_credentials 
-               = cli_credentials_init(tmp_ctx);
+       server_credentials = cli_credentials_init(tmp_ctx);
        if (!server_credentials) {
                DEBUG(1, ("Failed to init server credentials\n"));
-               return False;
+               return false;
        }
-       
-       cli_credentials_set_conf(server_credentials);
-       nt_status = cli_credentials_set_stored_principal(server_credentials, "kadmin/changepw");
+
+       /* 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);
+       nt_status = cli_credentials_set_stored_principal(server_credentials, kdc->task->event_ctx, kdc->task->lp_ctx, "kadmin/changepw");
        if (!NT_STATUS_IS_OK(nt_status)) {
                ret = kpasswdd_make_unauth_error_reply(kdc, mem_ctx, 
                                                       KRB5_KPASSWD_HARDERROR,
@@ -406,15 +490,35 @@ BOOL kpasswdd_process(struct kdc_server *kdc,
                return ret;
        }
        
-       gensec_set_credentials(gensec_security, server_credentials);
+       nt_status = gensec_set_credentials(gensec_security, server_credentials);
+       if (!NT_STATUS_IS_OK(nt_status)) {
+               talloc_free(tmp_ctx);
+               return false;
+       }
+
+       /* The kerberos PRIV packets include these addresses.  MIT
+        * clients check that they are present */
+       nt_status = gensec_set_peer_addr(gensec_security, peer_addr);
+       if (!NT_STATUS_IS_OK(nt_status)) {
+               talloc_free(tmp_ctx);
+               return false;
+       }
+       nt_status = gensec_set_my_addr(gensec_security, my_addr);
+       if (!NT_STATUS_IS_OK(nt_status)) {
+               talloc_free(tmp_ctx);
+               return false;
+       }
+
+       /* 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 False;
+               return false;
        }
 
+       /* Accept the AP-REQ and generate teh AP-REP we need for the reply */
        nt_status = gensec_update(gensec_security, tmp_ctx, ap_req, &ap_rep);
        if (!NT_STATUS_IS_OK(nt_status) && !NT_STATUS_EQUAL(nt_status, NT_STATUS_MORE_PROCESSING_REQUIRED)) {
                
@@ -432,6 +536,7 @@ BOOL kpasswdd_process(struct kdc_server *kdc,
                return ret;
        }
 
+       /* 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, 
@@ -448,15 +553,18 @@ BOOL kpasswdd_process(struct kdc_server *kdc,
                return ret;
        }
 
+       /* 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! */
-               return False;
+               return false;
        }
-       
+
+       /* 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)) {
@@ -477,7 +585,7 @@ BOOL kpasswdd_process(struct kdc_server *kdc,
 reply:
        *reply = data_blob_talloc(mem_ctx, NULL, krb_priv_rep.length + ap_rep.length + header_len);
        if (!reply->data) {
-               return False;
+               return false;
        }
 
        RSSVAL(reply->data, 0, reply->length);