]> eyrie.org Git - kerberos/krb5-strength.git/commitdiff
Change CrackLib tests for system CrackLib archive/debian/3.1-2
authorRuss Allbery <eagle@eyrie.org>
Sun, 25 Dec 2016 19:50:14 +0000 (11:50 -0800)
committerRuss Allbery <rra@debian.org>
Sat, 1 Sep 2018 00:07:43 +0000 (17:07 -0700)
The upstream test suite contains a few tests that fail with the
normal system CrackLib rules.  Modify them to expect passes so
that we can run the full test suite during the build.

Also remove CrackLib when testing character classes, since it
rejects the passwords used for minimum_different testing as too
simplistic.

Gbp-Pq: Name 0001-Change-CrackLib-tests-for-system-CrackLib.patch

tests/data/passwords/cracklib.json
tests/plugin/mit-t.c

index 037a70fdf5b9e01ea8a66cc96d9ed9178490b73c..b69e7005820a142475b679bdb6d0a7421a024b86 100644 (file)
@@ -22,9 +22,7 @@
     {
         "name": "in dictionary (repeated)",
         "principal": "test@EXAMPLE.ORG",
-        "password": "stanfordstanford",
-        "code": "KADM5_PASS_Q_GENERIC",
-        "error": "it is based on a (duplicated) dictionary word"
+        "password": "stanfordstanford"
     },
     {
         "name": "in dictionary (reversed)",
@@ -36,9 +34,7 @@
     {
         "name": "seven characters",
         "principal": "test@EXAMPLE.ORG",
-        "password": "dfareas",
-        "code": "KADM5_PASS_Q_GENERIC",
-        "error": "it is too short"
+        "password": "dfareas"
     },
     {
         "name": "four characters",
index 7eba594156091f927048869c99de21d5d3b74dd1..4406e38796e8475c5517c35b8dd70ddc5378ed97 100644 (file)
@@ -272,13 +272,13 @@ main(void)
     vtable->close(ctx, data);
 
     /* Add simple character class configuration to krb5.conf. */
-    setup_argv[5] = (char *) "minimum_different";
-    setup_argv[6] = (char *) "8";
-    setup_argv[7] = (char *) "require_ascii_printable";
+    setup_argv[3] = (char *) "minimum_different";
+    setup_argv[4] = (char *) "8";
+    setup_argv[5] = (char *) "require_ascii_printable";
+    setup_argv[6] = (char *) "true";
+    setup_argv[7] = (char *) "require_non_letter";
     setup_argv[8] = (char *) "true";
-    setup_argv[9] = (char *) "require_non_letter";
-    setup_argv[10] = (char *) "true";
-    setup_argv[11] = NULL;
+    setup_argv[9] = NULL;
     run_setup((const char **) setup_argv);
 
     /* Obtain a new Kerberos context with that krb5.conf file. */