r26639: librpc: Pass iconv convenience on from RPC connection to NDR library, so...
[bbaumbach/samba-autobuild/.git] / source4 / auth / kerberos / kerberos_pac.c
index b26a25eac18e3a1271a15696e653f7050a2817d1..d345ea969a346bb071b58e4876d2836775a8ac53 100644 (file)
@@ -10,7 +10,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,
 
    
    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 "system/kerberos.h"
-#include "system/time.h"
-#include "system/network.h"
-#include "auth/auth.h"
 #include "auth/kerberos/kerberos.h"
 #include "librpc/gen_ndr/ndr_krb5pac.h"
-#include "auth/auth.h"
-
-static NTSTATUS check_pac_checksum(TALLOC_CTX *mem_ctx, 
-                                  DATA_BLOB pac_data,
-                                  struct PAC_SIGNATURE_DATA *sig,
-                                  krb5_context context,
-                                  krb5_keyblock *keyblock)
+#include "lib/ldb/include/ldb.h"
+#include "auth/auth_sam_reply.h"
+#include "param/param.h"
+
+static krb5_error_code check_pac_checksum(TALLOC_CTX *mem_ctx, 
+                                         DATA_BLOB pac_data,
+                                         struct PAC_SIGNATURE_DATA *sig,
+                                         krb5_context context,
+                                         const krb5_keyblock *keyblock)
 {
        krb5_error_code ret;
        krb5_crypto crypto;
        Checksum cksum;
 
        cksum.cksumtype         = (CKSUMTYPE)sig->type;
-       cksum.checksum.length   = sizeof(sig->signature);
-       cksum.checksum.data     = sig->signature;
-
+       cksum.checksum.length   = sig->signature.length;
+       cksum.checksum.data     = sig->signature.data;
 
        ret = krb5_crypto_init(context,
                               keyblock,
@@ -55,7 +52,7 @@ static NTSTATUS check_pac_checksum(TALLOC_CTX *mem_ctx,
        if (ret) {
                DEBUG(0,("krb5_crypto_init() failed: %s\n", 
                          smb_get_krb5_error_message(context, ret, mem_ctx)));
-               return NT_STATUS_FOOBAR;
+               return ret;
        }
        ret = krb5_verify_checksum(context,
                                   crypto,
@@ -63,57 +60,104 @@ static NTSTATUS check_pac_checksum(TALLOC_CTX *mem_ctx,
                                   pac_data.data,
                                   pac_data.length,
                                   &cksum);
-       if (ret) {
-               DEBUG(2, ("PAC Verification failed: %s\n", 
-                         smb_get_krb5_error_message(context, ret, mem_ctx)));
-       }
-
        krb5_crypto_destroy(context, crypto);
 
-       if (ret) {
-               return NT_STATUS_ACCESS_DENIED;
-       }
-
-       return NT_STATUS_OK;
+       return ret;
 }
 
  NTSTATUS kerberos_decode_pac(TALLOC_CTX *mem_ctx,
                              struct PAC_DATA **pac_data_out,
                              DATA_BLOB blob,
                              krb5_context context,
-                             krb5_keyblock *krbtgt_keyblock,
-                             krb5_keyblock *service_keyblock)
+                             const krb5_keyblock *krbtgt_keyblock,
+                             const krb5_keyblock *service_keyblock,
+                             krb5_const_principal client_principal,
+                             time_t tgs_authtime,
+                             krb5_error_code *k5ret)
 {
+       krb5_error_code ret;
        NTSTATUS status;
-       struct PAC_SIGNATURE_DATA srv_sig;
+       enum ndr_err_code ndr_err;
        struct PAC_SIGNATURE_DATA *srv_sig_ptr = NULL;
-       struct PAC_SIGNATURE_DATA kdc_sig;
        struct PAC_SIGNATURE_DATA *kdc_sig_ptr = NULL;
+       struct PAC_SIGNATURE_DATA *srv_sig_wipe = NULL;
+       struct PAC_SIGNATURE_DATA *kdc_sig_wipe = NULL;
        struct PAC_LOGON_INFO *logon_info = NULL;
+       struct PAC_LOGON_NAME *logon_name = NULL;
        struct PAC_DATA *pac_data;
+       struct PAC_DATA_RAW *pac_data_raw;
+       struct smb_iconv_convenience *iconv_convenience = lp_iconv_convenience(global_loadparm);
+
+       DATA_BLOB *srv_sig_blob = NULL;
+       DATA_BLOB *kdc_sig_blob = NULL;
 
-       DATA_BLOB modified_pac_blob = data_blob_talloc(mem_ctx, blob.data, blob.length);
+       DATA_BLOB modified_pac_blob;
+       NTTIME tgs_authtime_nttime;
+       krb5_principal client_principal_pac;
        int i;
 
+       krb5_clear_error_string(context);
+
+       if (k5ret) {
+               *k5ret = KRB5_PARSE_MALFORMED;
+       }
+
        pac_data = talloc(mem_ctx, struct PAC_DATA);
-       if (!pac_data) {
+       pac_data_raw = talloc(mem_ctx, struct PAC_DATA_RAW);
+       kdc_sig_wipe = talloc(mem_ctx, struct PAC_SIGNATURE_DATA);
+       srv_sig_wipe = talloc(mem_ctx, struct PAC_SIGNATURE_DATA);
+       if (!pac_data_raw || !pac_data || !kdc_sig_wipe || !srv_sig_wipe) {
+               if (k5ret) {
+                       *k5ret = ENOMEM;
+               }
                return NT_STATUS_NO_MEMORY;
        }
 
-       status = ndr_pull_struct_blob(&blob, mem_ctx, pac_data,
-                                     (ndr_pull_flags_fn_t)ndr_pull_PAC_DATA);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(0,("can't parse the PAC\n"));
+       ndr_err = ndr_pull_struct_blob(&blob, pac_data, 
+                       iconv_convenience, pac_data,
+                      (ndr_pull_flags_fn_t)ndr_pull_PAC_DATA);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               status = ndr_map_error2ntstatus(ndr_err);
+               DEBUG(0,("can't parse the PAC: %s\n",
+                       nt_errstr(status)));
+               return status;
+       }
+
+       if (pac_data->num_buffers < 4) {
+               /* we need logon_ingo, service_key and kdc_key */
+               DEBUG(0,("less than 4 PAC buffers\n"));
+               return NT_STATUS_INVALID_PARAMETER;
+       }
+
+       ndr_err = ndr_pull_struct_blob(&blob, pac_data_raw, 
+                                      iconv_convenience, pac_data_raw,
+                                      (ndr_pull_flags_fn_t)ndr_pull_PAC_DATA_RAW);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               status = ndr_map_error2ntstatus(ndr_err);
+               DEBUG(0,("can't parse the PAC: %s\n",
+                       nt_errstr(status)));
                return status;
        }
 
-       if (pac_data->num_buffers < 3) {
+       if (pac_data_raw->num_buffers < 4) {
                /* we need logon_ingo, service_key and kdc_key */
-               DEBUG(0,("less than 3 PAC buffers\n"));
-               return NT_STATUS_FOOBAR;
+               DEBUG(0,("less than 4 PAC buffers\n"));
+               return NT_STATUS_INVALID_PARAMETER;
+       }
+
+       if (pac_data->num_buffers != pac_data_raw->num_buffers) {
+               /* we need logon_ingo, service_key and kdc_key */
+               DEBUG(0,("misparse!  PAC_DATA has %d buffers while PAC_DATA_RAW has %d\n",
+                        pac_data->num_buffers, pac_data_raw->num_buffers));
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
        for (i=0; i < pac_data->num_buffers; i++) {
+               if (pac_data->buffers[i].type != pac_data_raw->buffers[i].type) {
+                       DEBUG(0,("misparse!  PAC_DATA buffer %d has type %d while PAC_DATA_RAW has %d\n",
+                                i, pac_data->buffers[i].type, pac_data->buffers[i].type));
+                       return NT_STATUS_INVALID_PARAMETER;
+               }
                switch (pac_data->buffers[i].type) {
                        case PAC_TYPE_LOGON_INFO:
                                if (!pac_data->buffers[i].info) {
@@ -126,16 +170,17 @@ static NTSTATUS check_pac_checksum(TALLOC_CTX *mem_ctx,
                                        break;
                                }
                                srv_sig_ptr = &pac_data->buffers[i].info->srv_cksum;
-                               srv_sig = pac_data->buffers[i].info->srv_cksum;
+                               srv_sig_blob = &pac_data_raw->buffers[i].info->remaining;
                                break;
                        case PAC_TYPE_KDC_CHECKSUM:
                                if (!pac_data->buffers[i].info) {
                                        break;
                                }
                                kdc_sig_ptr = &pac_data->buffers[i].info->kdc_cksum;
-                               kdc_sig = pac_data->buffers[i].info->kdc_cksum;
+                               kdc_sig_blob = &pac_data_raw->buffers[i].info->remaining;
                                break;
                        case PAC_TYPE_LOGON_NAME:
+                               logon_name = &pac_data->buffers[i].info->logon_name;
                                break;
                        default:
                                break;
@@ -144,47 +189,141 @@ static NTSTATUS check_pac_checksum(TALLOC_CTX *mem_ctx,
 
        if (!logon_info) {
                DEBUG(0,("PAC no logon_info\n"));
-               return NT_STATUS_FOOBAR;
+               return NT_STATUS_INVALID_PARAMETER;
+       }
+
+       if (!logon_name) {
+               DEBUG(0,("PAC no logon_name\n"));
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
-       if (!srv_sig_ptr) {
+       if (!srv_sig_ptr || !srv_sig_blob) {
                DEBUG(0,("PAC no srv_key\n"));
-               return NT_STATUS_FOOBAR;
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
-       if (!kdc_sig_ptr) {
+       if (!kdc_sig_ptr || !kdc_sig_blob) {
                DEBUG(0,("PAC no kdc_key\n"));
-               return NT_STATUS_FOOBAR;
+               return NT_STATUS_INVALID_PARAMETER;
        }
 
-       memset(&modified_pac_blob.data[modified_pac_blob.length - 20],
-              '\0', 16);
-       memset(&modified_pac_blob.data[modified_pac_blob.length - 44],
-              '\0', 16);
+       /* Find and zero out the signatures, as required by the signing algorithm */
 
-       /* verify by service_key */
-       status = check_pac_checksum(mem_ctx, 
-                                   modified_pac_blob, &srv_sig, 
-                                   context, 
-                                   service_keyblock);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(1, ("PAC Decode: Failed to verify the service signature\n"));
+       /* We find the data blobs above, now we parse them to get at the exact portion we should zero */
+       ndr_err = ndr_pull_struct_blob(kdc_sig_blob, kdc_sig_wipe, 
+                                      iconv_convenience, kdc_sig_wipe,
+                                      (ndr_pull_flags_fn_t)ndr_pull_PAC_SIGNATURE_DATA);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               status = ndr_map_error2ntstatus(ndr_err);
+               DEBUG(0,("can't parse the KDC signature: %s\n",
+                       nt_errstr(status)));
+               return status;
+       }
+       
+       ndr_err = ndr_pull_struct_blob(srv_sig_blob, srv_sig_wipe, 
+                                      iconv_convenience, srv_sig_wipe,
+                                      (ndr_pull_flags_fn_t)ndr_pull_PAC_SIGNATURE_DATA);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               status = ndr_map_error2ntstatus(ndr_err);
+               DEBUG(0,("can't parse the SRV signature: %s\n",
+                       nt_errstr(status)));
                return status;
        }
 
-       if (krbtgt_keyblock) {
-               DATA_BLOB service_checksum_blob
-                       = data_blob_const(srv_sig_ptr->signature, sizeof(srv_sig_ptr->signature));
+       /* Now zero the decoded structure */
+       memset(kdc_sig_wipe->signature.data, '\0', kdc_sig_wipe->signature.length);
+       memset(srv_sig_wipe->signature.data, '\0', srv_sig_wipe->signature.length);
+       
+       /* and reencode, back into the same place it came from */
+       ndr_err = ndr_push_struct_blob(kdc_sig_blob, pac_data_raw, 
+                                      iconv_convenience,
+                                      kdc_sig_wipe,
+                                      (ndr_push_flags_fn_t)ndr_push_PAC_SIGNATURE_DATA);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               status = ndr_map_error2ntstatus(ndr_err);
+               DEBUG(0,("can't repack the KDC signature: %s\n",
+                       nt_errstr(status)));
+               return status;
+       }
+       ndr_err = ndr_push_struct_blob(srv_sig_blob, pac_data_raw, 
+                                      iconv_convenience,
+                                      srv_sig_wipe,
+                                      (ndr_push_flags_fn_t)ndr_push_PAC_SIGNATURE_DATA);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               status = ndr_map_error2ntstatus(ndr_err);
+               DEBUG(0,("can't repack the SRV signature: %s\n",
+                       nt_errstr(status)));
+               return status;
+       }
+
+       /* push out the whole structure, but now with zero'ed signatures */
+       ndr_err = ndr_push_struct_blob(&modified_pac_blob, pac_data_raw, 
+                                      iconv_convenience,
+                                      pac_data_raw,
+                                      (ndr_push_flags_fn_t)ndr_push_PAC_DATA_RAW);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               status = ndr_map_error2ntstatus(ndr_err);
+               DEBUG(0,("can't repack the RAW PAC: %s\n",
+                       nt_errstr(status)));
+               return status;
+       }
+
+       /* verify by service_key */
+       ret = check_pac_checksum(mem_ctx, 
+                                modified_pac_blob, srv_sig_ptr, 
+                                context, 
+                                service_keyblock);
+       if (ret) {
+               DEBUG(1, ("PAC Decode: Failed to verify the service signature: %s\n",
+                         smb_get_krb5_error_message(context, ret, mem_ctx)));
+               if (k5ret) {
+                       *k5ret = ret;
+               }
+               return NT_STATUS_ACCESS_DENIED;
+       }
 
-               status = check_pac_checksum(mem_ctx, 
-                                           service_checksum_blob, &kdc_sig, 
+       if (krbtgt_keyblock) {
+               ret = check_pac_checksum(mem_ctx, 
+                                           srv_sig_ptr->signature, kdc_sig_ptr, 
                                            context, krbtgt_keyblock);
-               if (!NT_STATUS_IS_OK(status)) {
-                       DEBUG(1, ("PAC Decode: Failed to verify the krbtgt signature\n"));
-                       return status;
+               if (ret) {
+                       DEBUG(1, ("PAC Decode: Failed to verify the KDC signature: %s\n",
+                                 smb_get_krb5_error_message(context, ret, mem_ctx)));
+                       if (k5ret) {
+                               *k5ret = ret;
+                       }
+                       return NT_STATUS_ACCESS_DENIED;
                }
        }
 
+       /* Convert to NT time, so as not to loose accuracy in comparison */
+       unix_to_nt_time(&tgs_authtime_nttime, tgs_authtime);
+
+       if (tgs_authtime_nttime != logon_name->logon_time) {
+               DEBUG(2, ("PAC Decode: Logon time mismatch between ticket and PAC!\n"));
+               DEBUG(2, ("PAC Decode: PAC: %s\n", nt_time_string(mem_ctx, logon_name->logon_time)));
+               DEBUG(2, ("PAC Decode: Ticket: %s\n", nt_time_string(mem_ctx, tgs_authtime_nttime)));
+               return NT_STATUS_ACCESS_DENIED;
+       }
+
+       ret = krb5_parse_name_flags(context, logon_name->account_name, KRB5_PRINCIPAL_PARSE_NO_REALM, 
+                                   &client_principal_pac);
+       if (ret) {
+               DEBUG(2, ("Could not parse name from incoming PAC: [%s]: %s\n", 
+                         logon_name->account_name, 
+                         smb_get_krb5_error_message(context, ret, mem_ctx)));
+               if (k5ret) {
+                       *k5ret = ret;
+               }
+               return NT_STATUS_INVALID_PARAMETER;
+       }
+
+       if (!krb5_principal_compare_any_realm(context, client_principal, client_principal_pac)) {
+               DEBUG(2, ("Name in PAC [%s] does not match principal name in ticket\n", 
+                         logon_name->account_name));
+               return NT_STATUS_ACCESS_DENIED;
+       }
+       
 #if 0
        if (strcasecmp(logon_info->info3.base.account_name.string, 
                       "Administrator")== 0) {
@@ -192,30 +331,35 @@ static NTSTATUS check_pac_checksum(TALLOC_CTX *mem_ctx,
        }
 #endif
 
-       DEBUG(0,("account_name: %s [%s]\n",
+       DEBUG(3,("Found account name from PAC: %s [%s]\n",
                 logon_info->info3.base.account_name.string, 
                 logon_info->info3.base.full_name.string));
        *pac_data_out = pac_data;
 
-       return status;
+       return NT_STATUS_OK;
 }
 
- NTSTATUS kerberos_pac_logon_info(TALLOC_CTX *mem_ctx,
+_PUBLIC_  NTSTATUS kerberos_pac_logon_info(TALLOC_CTX *mem_ctx,
                                  struct PAC_LOGON_INFO **logon_info,
                                  DATA_BLOB blob,
                                  krb5_context context,
-                                 krb5_keyblock *krbtgt_keyblock,
-                                 krb5_keyblock *service_keyblock)
+                                 const krb5_keyblock *krbtgt_keyblock,
+                                 const krb5_keyblock *service_keyblock,
+                                 krb5_const_principal client_principal,
+                                 time_t tgs_authtime, 
+                                 krb5_error_code *k5ret)
 {
        NTSTATUS nt_status;
        struct PAC_DATA *pac_data;
        int i;
-
        nt_status = kerberos_decode_pac(mem_ctx, &pac_data,
                                        blob,
                                        context,
                                        krbtgt_keyblock,
-                                       service_keyblock);
+                                       service_keyblock,
+                                       client_principal, 
+                                       tgs_authtime,
+                                       k5ret);
        if (!NT_STATUS_IS_OK(nt_status)) {
                return nt_status;
        }
@@ -237,7 +381,7 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
                                         DATA_BLOB *pac_data,
                                         struct PAC_SIGNATURE_DATA *sig,
                                         krb5_context context,
-                                        krb5_keyblock *keyblock)
+                                        const krb5_keyblock *keyblock)
 {
        krb5_error_code ret;
        krb5_crypto crypto;
@@ -272,9 +416,8 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
        }
 
        sig->type = cksum.cksumtype;
-       if (cksum.checksum.length == sizeof(sig->signature)) {
-               memcpy(sig->signature, cksum.checksum.data, sizeof(sig->signature));
-       }
+       sig->signature = data_blob_talloc(mem_ctx, cksum.checksum.data, cksum.checksum.length);
+       free_Checksum(&cksum);
 
        return 0;
 }
@@ -282,15 +425,15 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
  krb5_error_code kerberos_encode_pac(TALLOC_CTX *mem_ctx,
                                    struct PAC_DATA *pac_data,
                                    krb5_context context,
-                                   krb5_keyblock *krbtgt_keyblock,
-                                   krb5_keyblock *service_keyblock,
+                                   const krb5_keyblock *krbtgt_keyblock,
+                                   const krb5_keyblock *service_keyblock,
                                    DATA_BLOB *pac) 
 {
        NTSTATUS nt_status;
        krb5_error_code ret;
+       enum ndr_err_code ndr_err;
        DATA_BLOB zero_blob = data_blob(NULL, 0);
        DATA_BLOB tmp_blob = data_blob(NULL, 0);
-       DATA_BLOB service_checksum_blob;
        struct PAC_SIGNATURE_DATA *kdc_checksum = NULL;
        struct PAC_SIGNATURE_DATA *srv_checksum = NULL;
        int i;
@@ -338,12 +481,15 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
        }
 
        /* But wipe out the actual signatures */
-       ZERO_STRUCT(kdc_checksum->signature);
-       ZERO_STRUCT(srv_checksum->signature);
-
-       nt_status = ndr_push_struct_blob(&tmp_blob, mem_ctx, pac_data,
-                                        (ndr_push_flags_fn_t)ndr_push_PAC_DATA);
-       if (!NT_STATUS_IS_OK(nt_status)) {
+       memset(kdc_checksum->signature.data, '\0', kdc_checksum->signature.length);
+       memset(srv_checksum->signature.data, '\0', srv_checksum->signature.length);
+
+       ndr_err = ndr_push_struct_blob(&tmp_blob, mem_ctx, 
+                                      lp_iconv_convenience(global_loadparm),
+                                      pac_data,
+                                      (ndr_push_flags_fn_t)ndr_push_PAC_DATA);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               nt_status = ndr_map_error2ntstatus(ndr_err);
                DEBUG(1, ("PAC (presig) push failed: %s\n", nt_errstr(nt_status)));
                talloc_free(pac_data);
                return EINVAL;
@@ -353,11 +499,8 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
        ret = make_pac_checksum(mem_ctx, &tmp_blob, srv_checksum,
                                context, service_keyblock);
 
-       service_checksum_blob
-               = data_blob_const(srv_checksum->signature, sizeof(srv_checksum->signature));
-
        /* Then sign Server checksum */
-       ret = make_pac_checksum(mem_ctx, &service_checksum_blob, kdc_checksum, context, krbtgt_keyblock);
+       ret = make_pac_checksum(mem_ctx, &srv_checksum->signature, kdc_checksum, context, krbtgt_keyblock);
        if (ret) {
                DEBUG(2, ("making krbtgt PAC checksum failed: %s\n", 
                          smb_get_krb5_error_message(context, ret, mem_ctx)));
@@ -366,9 +509,12 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
        }
 
        /* And push it out again, this time to the world.  This relies on determanistic pointer values */
-       nt_status = ndr_push_struct_blob(&tmp_blob, mem_ctx, pac_data,
-                                        (ndr_push_flags_fn_t)ndr_push_PAC_DATA);
-       if (!NT_STATUS_IS_OK(nt_status)) {
+       ndr_err = ndr_push_struct_blob(&tmp_blob, mem_ctx, 
+                                      lp_iconv_convenience(global_loadparm),
+                                      pac_data,
+                                      (ndr_push_flags_fn_t)ndr_push_PAC_DATA);
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               nt_status = ndr_map_error2ntstatus(ndr_err);
                DEBUG(1, ("PAC (final) push failed: %s\n", nt_errstr(nt_status)));
                talloc_free(pac_data);
                return EINVAL;
@@ -383,8 +529,9 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
  krb5_error_code kerberos_create_pac(TALLOC_CTX *mem_ctx,
                                     struct auth_serversupplied_info *server_info,
                                     krb5_context context,
-                                    krb5_keyblock *krbtgt_keyblock,
-                                    krb5_keyblock *service_keyblock,
+                                    const krb5_keyblock *krbtgt_keyblock,
+                                    const krb5_keyblock *service_keyblock,
+                                    krb5_principal client_principal,
                                     time_t tgs_authtime,
                                     DATA_BLOB *pac)
 {
@@ -392,7 +539,6 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
        krb5_error_code ret;
        struct PAC_DATA *pac_data = talloc(mem_ctx, struct PAC_DATA);
        struct netr_SamInfo3 *sam3;
-       struct timeval tv = timeval_current();
        union PAC_INFO *u_LOGON_INFO;
        struct PAC_LOGON_INFO *LOGON_INFO;
        union PAC_INFO *u_LOGON_NAME;
@@ -400,6 +546,8 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
        union PAC_INFO *u_KDC_CHECKSUM;
        union PAC_INFO *u_SRV_CHECKSUM;
 
+       char *name;
+               
        enum {
                PAC_BUF_LOGON_INFO = 0,
                PAC_BUF_LOGON_NAME = 1,
@@ -476,13 +624,17 @@ static krb5_error_code make_pac_checksum(TALLOC_CTX *mem_ctx,
 
        u_LOGON_INFO->logon_info.info           = LOGON_INFO;
        LOGON_INFO->info3 = *sam3;
-       LOGON_INFO->info3.base.last_logon       = timeval_to_nttime(&tv);
-
-       LOGON_NAME->account_name        = server_info->account_name;
 
+       ret = krb5_unparse_name_flags(context, client_principal, 
+                                     KRB5_PRINCIPAL_UNPARSE_NO_REALM, &name);
+       if (ret) {
+               return ret;
+       }
+       LOGON_NAME->account_name        = talloc_strdup(LOGON_NAME, name);
+       free(name);
        /*
          this logon_time field is absolutely critical. This is what
-         caused all our pac troubles :-)
+         caused all our PAC troubles :-)
        */
        unix_to_nt_time(&LOGON_NAME->logon_time, tgs_authtime);