s4:torture/ndr: make use of torture_suite_add_ndr_pull_validate_test() in krb5pac...
authorStefan Metzmacher <metze@samba.org>
Tue, 19 Jul 2016 14:38:56 +0000 (16:38 +0200)
committerStefan Metzmacher <metze@samba.org>
Wed, 20 Jul 2016 19:27:19 +0000 (21:27 +0200)
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
source4/torture/ndr/krb5pac.c

index 1468285842a3a806e446930a1dd23484c827bb52..def444ddf019ee5359d18c5635e466a01fd81a2e 100644 (file)
@@ -302,25 +302,47 @@ struct torture_suite *ndr_krb5pac_suite(TALLOC_CTX *ctx)
 {
        struct torture_suite *suite = torture_suite_create(ctx, "krb5pac");
 
-       torture_suite_add_ndr_pull_test(suite,
+       torture_suite_add_ndr_pull_validate_test(suite,
                                        PAC_DATA_RAW,
-                                       PAC_DATA_data,
+                                       data_blob_const(PAC_DATA_data,
+                                                       sizeof(PAC_DATA_data)),
                                        NULL);
+       /*
+        * We can't use torture_suite_add_ndr_pull_validate_test()
+        * here with PAC_DATA, as we don't match the unique
+        * pointer values inside PAC_LOGON_INFO, for these
+        * case where we have S-1-5-18-1, as extra sid.
+        */
        torture_suite_add_ndr_pull_test(suite,
                                        PAC_DATA,
                                        PAC_DATA_data,
                                        PAC_DATA_check);
-       torture_suite_add_ndr_pull_test(suite,
+
+       torture_suite_add_ndr_pull_validate_test(suite,
                                        PAC_DATA_RAW,
-                                       PAC_DATA_data2,
+                                       data_blob_const(PAC_DATA_data2,
+                                                       sizeof(PAC_DATA_data2)),
                                        NULL);
+       /*
+        * We can't use torture_suite_add_ndr_pull_validate_test()
+        * here with PAC_DATA, as we don't match the unique
+        * pointer values inside PAC_LOGON_INFO, for these
+        * case where we have S-1-5-18-1, as extra sid.
+        */
        torture_suite_add_ndr_pull_test(suite,
                                        PAC_DATA,
                                        PAC_DATA_data2,
                                        PAC_DATA_check2);
-       torture_suite_add_ndr_pull_test(suite,
+
+       torture_suite_add_ndr_pull_validate_test(suite,
                                        PAC_DATA_RAW,
-                                       PAC_DATA_data3,
+                                       data_blob_const(PAC_DATA_data3,
+                                                       sizeof(PAC_DATA_data3)),
+                                       NULL);
+       torture_suite_add_ndr_pull_validate_test(suite,
+                                       PAC_DATA,
+                                       data_blob_const(PAC_DATA_data3,
+                                                       sizeof(PAC_DATA_data3)),
                                        NULL);
 
        return suite;