r8792: Clarify comments
authorAndrew Bartlett <abartlet@samba.org>
Wed, 27 Jul 2005 00:25:17 +0000 (00:25 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 18:30:06 +0000 (13:30 -0500)
(This used to be commit 6174cf826fac8288b6604d9140b741aab2156cb3)

source4/torture/auth/pac.c

index 21c3119e31679bade56383873402db778ef86a83..59bf00897069ef0eb46b61766e460c6737119007 100644 (file)
@@ -215,6 +215,9 @@ static BOOL torture_pac_saved_check(void)
                return False;
        }
 
+       /* The krbtgt key in use when the above PAC was generated.
+        * This is an arcfour-hmac-md5 key, extracted with our 'net
+        * samdump' tool. */
        krbtgt_bytes = smbpasswd_gethexpwd(mem_ctx, "B286757148AF7FD252C53603A150B7E7");
        if (!krbtgt_bytes) {
                DEBUG(0, ("Could not interpret krbtgt key"));
@@ -296,10 +299,9 @@ static BOOL torture_pac_saved_check(void)
 
        dump_data(10,validate_blob.data,validate_blob.length);
 
-       /* all we can check is the length of the buffers,
-        * to check that the alignment and padding is ok,
-        * we can't compare the bytes, because we use a different algorithm
-        * to create the pointer values
+       /* compare both the length and the data bytes after a
+        * pull/push cycle.  This ensures we use the exact same
+        * pointer, padding etc algorithms as win2k3.
         */
        if (tmp_blob.length != validate_blob.length) {
                DEBUG(0, ("PAC push failed: orignial buffer length[%u] != created buffer length[%u]\n",