svn commit: samba r8001 - in branches/SAMBA_4_0/source: auth/kerberos kdc librpc/idl

[abartlet at samba.org]

Author: abartlet
Date: 2005-06-30 01:04:51 +0000 (Thu, 30 Jun 2005)
New Revision: 8001

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=8001

Log:
Also fill in the krbtgt checksum, and make sure to put the right
checksum in the right place...

Andrew Bartlett

Modified:
   branches/SAMBA_4_0/source/auth/kerberos/kerberos.h
   branches/SAMBA_4_0/source/auth/kerberos/kerberos_pac.c
   branches/SAMBA_4_0/source/kdc/pac-glue.c
   branches/SAMBA_4_0/source/librpc/idl/krb5pac.idl


Changeset:
Modified: branches/SAMBA_4_0/source/auth/kerberos/kerberos.h
===================================================================
--- branches/SAMBA_4_0/source/auth/kerberos/kerberos.h	2005-06-29 22:31:45 UTC (rev 8000)
+++ branches/SAMBA_4_0/source/auth/kerberos/kerberos.h	2005-06-30 01:04:51 UTC (rev 8001)
@@ -132,10 +132,12 @@
 			     DATA_BLOB blob,
 			     struct smb_krb5_context *smb_krb5_context,
 			     krb5_keyblock *keyblock);
+
 krb5_error_code kerberos_encode_pac(TALLOC_CTX *mem_ctx,
 				    struct auth_serversupplied_info *server_info,
 				    krb5_context context,
-				    krb5_keyblock *keyblock,
+				    krb5_keyblock *krbtgt_keyblock,
+				    krb5_keyblock *server_keyblock,
 				    krb5_data *pac);
 #endif /* HAVE_KRB5 */
 

Modified: branches/SAMBA_4_0/source/auth/kerberos/kerberos_pac.c
===================================================================
--- branches/SAMBA_4_0/source/auth/kerberos/kerberos_pac.c	2005-06-29 22:31:45 UTC (rev 8000)
+++ branches/SAMBA_4_0/source/auth/kerberos/kerberos_pac.c	2005-06-30 01:04:51 UTC (rev 8001)
@@ -222,11 +222,13 @@
  krb5_error_code kerberos_encode_pac(TALLOC_CTX *mem_ctx,
 				     struct auth_serversupplied_info *server_info,
 				     krb5_context context,
-				     krb5_keyblock *keyblock,
+				     krb5_keyblock *krbtgt_keyblock,
+				     krb5_keyblock *server_keyblock,
 				     krb5_data *pac)
 {
 	NTSTATUS nt_status;
 	DATA_BLOB tmp_blob = data_blob(NULL, 0);
+	DATA_BLOB server_checksum_blob;
 	krb5_error_code ret;
 	struct PAC_DATA *pac_data = talloc(mem_ctx, struct PAC_DATA);
 	struct netr_SamBaseInfo *sam;
@@ -279,7 +281,10 @@
 	
 	/* First, just get the keytypes filled in (and lengths right, eventually) */
 	ret = make_pac_checksum(mem_ctx, tmp_blob, &pac_data->buffers[2].info->srv_cksum,
-				context, keyblock);
+				context, krbtgt_keyblock);
+
+	ret = make_pac_checksum(mem_ctx, tmp_blob, &pac_data->buffers[3].info->srv_cksum,
+				context, server_keyblock);
 	if (ret) {
 		DEBUG(2, ("making PAC checksum failed: %s\n", 
 			  smb_get_krb5_error_message(context, ret, mem_ctx)));
@@ -303,9 +308,22 @@
 
 	/* Then sign the result of the previous push, where the sig was zero'ed out */
 	ret = make_pac_checksum(mem_ctx, tmp_blob, &pac_data->buffers[3].info->srv_cksum,
-				context, keyblock);
+				context, server_keyblock);
 
-	/* And push it out to the world.  This relies on determanistic pointer values */
+	/* Push the Server checksum out */
+	nt_status = ndr_push_struct_blob(&server_checksum_blob, mem_ctx, &pac_data->buffers[3].info->srv_cksum,
+					 (ndr_push_flags_fn_t)ndr_push_PAC_SIGNATURE_DATA);
+	if (!NT_STATUS_IS_OK(nt_status)) {
+		DEBUG(1, ("PAC push failed: %s\n", nt_errstr(nt_status)));
+		talloc_free(pac_data);
+		return EINVAL;
+	}
+
+	/* Then sign the result of the previous push, where the sig was zero'ed out */
+	ret = make_pac_checksum(mem_ctx, server_checksum_blob, &pac_data->buffers[2].info->kdc_cksum,
+				context, krbtgt_keyblock);
+
+	/* 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)) {

Modified: branches/SAMBA_4_0/source/kdc/pac-glue.c
===================================================================
--- branches/SAMBA_4_0/source/kdc/pac-glue.c	2005-06-29 22:31:45 UTC (rev 8000)
+++ branches/SAMBA_4_0/source/kdc/pac-glue.c	2005-06-30 01:04:51 UTC (rev 8001)
@@ -27,7 +27,8 @@
  krb5_error_code samba_get_pac(krb5_context context, 
 			      struct krb5_kdc_configuration *config,
 			      krb5_principal client, 
-			      krb5_keyblock *keyblock, 
+			      krb5_keyblock *krbtgt_keyblock, 
+			      krb5_keyblock *server_keyblock, 
 			      krb5_data *pac) 
 {
 	krb5_error_code ret;
@@ -64,13 +65,13 @@
 	if (!NT_STATUS_IS_OK(nt_status)) {
 		DEBUG(0, ("Getting user info for PAC failed: %s\n",
 			  nt_errstr(nt_status)));
-		talloc_free(mem_ctx);
 		return EINVAL;
 	}
 
 	ret = kerberos_encode_pac(mem_ctx, server_info, 
 				  context, 
-				  keyblock,
+				  krbtgt_keyblock,
+				  server_keyblock,
 				  pac);
 
 	talloc_free(mem_ctx);

Modified: branches/SAMBA_4_0/source/librpc/idl/krb5pac.idl
===================================================================
--- branches/SAMBA_4_0/source/librpc/idl/krb5pac.idl	2005-06-29 22:31:45 UTC (rev 8000)
+++ branches/SAMBA_4_0/source/librpc/idl/krb5pac.idl	2005-06-30 01:04:51 UTC (rev 8001)
@@ -19,7 +19,7 @@
 		[value(0)] uint32 _pad; 
 	} PAC_LOGON_NAME;
 
-	typedef [flag(NDR_PAHEX)] struct {
+	typedef [public,flag(NDR_PAHEX)] struct {
 		uint32 type;
 		uint8 signature[16];
 		[value(0)] uint32 _pad;