r8252: Steal metze's thunder, and prove that with a few small tweaks, we can

now push/pull a sample PAC, and still have the same byte buffer.
(Metze set up the string code, and probably already has a similar
patch).

Unfortunetly win2k3 still doesn't like what we provide, but every step helps.

Also use data_blob_const() when we are just wrapping data for API
reasons.

Andrew Bartlett
(This used to be commit e7c8076fc1459ff2ccefdaf0b091d04ee6137957)

1 files changed, 8 insertions, 1 deletions

diff --git a/source4/torture/auth/pac.c b/source4/torture/auth/pac.c
index ade68fcd77..f03b20b286 100644
--- a/source4/torture/auth/pac.c
+++ b/source4/torture/auth/pac.c
@@ -302,12 +302,19 @@ static BOOL torture_pac_saved_check(void)
 	 * to create the pointer values
 	 */
 	if (tmp_blob.length != validate_blob.length) {
-		DEBUG(0, ("PAC push failed orignial buffer length[%u] != created buffer length[%u]\n",
+		DEBUG(0, ("PAC push failed: orignial buffer length[%u] != created buffer length[%u]\n",
 				tmp_blob.length, validate_blob.length));
 		talloc_free(mem_ctx);
 		return False;
 	}
 
+	if (memcmp(tmp_blob.data, validate_blob.data, tmp_blob.length) != 0) {
+		DEBUG(0, ("PAC push failed: length[%u] matches, but data does not\n",
+			  tmp_blob.length));
+		talloc_free(mem_ctx);
+		return False;
+	}
+
 	talloc_free(mem_ctx);
 	return True;
 }