r8148: - make the PAC generation code a bit more readable and add some outof memory checks

- move to handmodified pull/push code for PAC_BUFFER
  to get the _ndr_size field and the subcontext size right

- after looking closely to the sample w2k3 PAC in our torture test (and some more in my archive)
  I found out that the first uint32 before the netr_SamInfo3 was also a pointer,
  (and we passed a NULL pointer there before, so I think that was the reason why the windows clients doesn't want our PAC)

  w2k3 uses this for unique pointers:

  ptr = ndr->ptr_count * 4;
  ptr |= 0x00020000;
  ndr->ptr_count;

- do one more pull/push round with the sample PAC

metze
(This used to be commit 0eee17941595e9842a264bf89ac73ca66cea7ed5)

1 files changed, 26 insertions, 2 deletions

diff --git a/source4/torture/auth/pac.c b/source4/torture/auth/pac.c
index 00196b493f..51f33781fa 100644
--- a/source4/torture/auth/pac.c
+++ b/source4/torture/auth/pac.c
@@ -118,7 +118,9 @@ static BOOL torture_pac_self_check(void)
 		talloc_free(mem_ctx);
 		return False;
 	}
-	
+
+	/* dump_data(0,tmp_blob.data,tmp_blob.length); */
+
 	/* Now check that we can read it back */
 	nt_status = kerberos_decode_pac(mem_ctx, &pac_info,
 					tmp_blob,
@@ -190,8 +192,9 @@ static BOOL torture_pac_saved_check(void)
 {
 	NTSTATUS nt_status;
 	TALLOC_CTX *mem_ctx = talloc_named(NULL, 0, "PAC saved check");
-	DATA_BLOB tmp_blob;
+	DATA_BLOB tmp_blob, validate_blob;
 	struct PAC_LOGON_INFO *pac_info;
+	struct PAC_DATA pac_data;
 	krb5_keyblock server_keyblock;
 	uint8_t server_bytes[16];
 	
@@ -225,6 +228,10 @@ static BOOL torture_pac_saved_check(void)
 
 	tmp_blob = data_blob_const(saved_pac, sizeof(saved_pac));
 
+	/*tmp_blob.data = file_load(lp_parm_string(-1,"torture","pac_file"), &tmp_blob.length);*/
+
+	/*dump_data(0,tmp_blob.data,tmp_blob.length);*/
+
 	/* Decode and verify the signaure on the PAC */
 	nt_status = kerberos_decode_pac(mem_ctx, &pac_info,
 					tmp_blob,
@@ -239,6 +246,23 @@ static BOOL torture_pac_saved_check(void)
 		talloc_free(mem_ctx);
 		return False;
 	}
+
+	nt_status = ndr_pull_struct_blob(&tmp_blob, mem_ctx, &pac_data,
+					 (ndr_pull_flags_fn_t)ndr_pull_PAC_DATA);
+	if (!NT_STATUS_IS_OK(nt_status)) {
+		DEBUG(0,("can't parse the PAC\n"));
+		return False;
+	}
+
+	nt_status = ndr_push_struct_blob(&validate_blob, mem_ctx, &pac_data,
+					 (ndr_push_flags_fn_t)ndr_push_PAC_DATA);
+	if (!NT_STATUS_IS_OK(nt_status)) {
+		DEBUG(0, ("PAC push failed: %s\n", nt_errstr(nt_status)));
+		return False;
+	}
+
+	/* dump_data(0,validate_blob.data,validate_blob.length); */
+
 	talloc_free(mem_ctx);
 	return True;
 }