Change CrackLib tests for system CrackLib

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

Finalize changes for 3.1-2

Remove obsolete debian/source/options

* Remove obsolete debian/source/options that was forcing the compression
  format to xz (now the default).

Add upstream-vcs-tag configuration to debian/gbp.conf

Refresh upstream signing key

Remove trailing whitespace in debian/changelog

Update to debhelper compatibility levl V11

Bump debian/watch version to 4 and use https

Update standards version to 4.2.1

* Update standards version to 4.2.1.
  - Enable verbose test output.
  - Install the upstream release notes as NEWS.gz, not changelog.gz.
  - Add Rules-Requires-Root: no.
  - Use https for URLs in debian/copyright.
  - Change priority to optional.

Remove another stray --parallel flag

Finalize changes for 3.1-1

Replace missing PACKAGE setting in debian/rules

Was too aggressive about removing this.

Fix test suite results with system CrackLib

* Patch the upstream test suite to change the expected results for a few
  passwords that are rejected by the embedded CrackLib but accepted by
  the system CrackLib (which the Debian package is built with).

Update to debhelper compatibility level V10

* Update to debhelper compatibility level V10.
  - Remove explicit dh-autoreconf dependency and invocation.
  - Remove explicit --parallel flags.

Update standards version to 3.9.8

Enable all hardening flags

Refresh upstream signing key

Refresh debian/copyright for new upstream release

Run wrap-and-sort -ast

Switch to https for all package URLs

Re-enable treating test suite failures as build failures

* Re-enable treating test suite failures as package build failures now
  that bug #724570 in CrackLib was fixed in Debian.

Add changelog for new upstream release

Merge tag 'upstream/3.1' into debian/master

Upstream version 3.1

New upstream version 3.1

Switch to the DEP-14 branch layout

* Switch to the DEP-14 branch layout and update debian/gbp.conf and
  Vcs-Git accordingly.

Update files included in the distribution

Release 3.1

Add NEWS entry for require_classes of a number of classes

Coding style cleanup and tests for minimum classes

Add tests for specifying a minimum number of classes, refactor for
coding style a bit, and add new tests for the new syntax errors.

Add support for requiring a number of character classes

Extend the required_classes config option to allow requiring at
least a given number of different character classes (whatever they
are).

Handle errors in heimdal-strength tests better

In Travis CI, the heimdal-strength test was failing with a broken
pipe.  Apparently the timing was such that the child process would
exit with an error before IPC::Run could try to write the input,
and then IPC::Run would die with a broken pipe.  Work around this
by allowing a flag to be passed to run_heimdal_strength saying whether
to expect an error, and don't send input in that case.

Add DocKnot metadata and regenerate README

This also adds a Markdown version of README, mostly for GitHub.

Move configuration instructions to man pages

Create a new krb5-strength man page that gets the configuration
instructions for the plugins (with the proper path substituted in
by the Makefile) and move other configuration details to the
heimdal-strength and heimdal-history man pages.  Duplicate the
documentation for cracklib_maxlen in the heimdal-strength man page.

This will make it easier to automate generation of the README file,
since it will now require less complex formatting.

Fix some sorting bugs in embedded CrackLib

Patch the mkdict and packer in the embedded copy of CrackLib to force
C locale when sorting (avoiding a corrupted dictionary) and warn and
skip out-of-order words rather than creating a corrupted dictionary.
Patch from Mark Sirota.

Also check cracklib_maxlen with heimdal-strength

Finalize Travis-CI configuration

Now that all the bugs are fixed.  (Probably should have done that
in a branch with a pull request.  Oh well.)

One more attempt to fix perltidy on Travis-CI

Hopefully fix perltidy failure in Travis-CI

More Travis-CI debugging of test failure

Temporarily run perl/critic test verbose for Travis

Need to figure out why this is failing.

Add Travis-CI configuration

Update to rra-c-util 5.7 and C TAP Harness 4.1

Update to rra-c-util 6.2:

* Use calloc in preference to malloc wherever appropriate.
* Use reallocarray in preference to realloc wherever appropriate.
* Suppress warnings from Kerberos headers under make warnings.
* Support the embedded Kerberos in Solaris 10 in library probes.
* Add missing va_end in xasprintf implementation.
* Fix logic in Test::RRA::Automake for new Automake dist checking.
* Fix all return-value checks for snprintf to avoid off-by-one error.
* Update warning flags for make warnings to GCC 6.1.0.
* Fix Test::RRA::Config for new "do" semantics in Perl 5.22.2.
* Add a new test for obsolete eyrie.org URLs.
* Require Test::Strict 0.25 or newer for Perl strictness checks.

Update to C TAP Harness 4.1:

* Replace all remaining uses of sprintf.
* Test lists may now have comments and blank lines.
* runtests -v will show the complete output from a test.
* Fix segfault in runtests when given an empty test list.
* Tests use C_TAP_SOURCE and C_TAP_BUILD instead of SOURCE and BUILD.

Fix Perl critic error in tests/tools/wordlist-t

Don't pass $_ explicitly as an argument to length().

Rename autogen to bootstrap

Rename the script to bootstrap from a Git checkout to bootstrap,
matching the emerging consensus in the Autoconf world.

Add NEWS, documentation, and test suite for cracklib_maxlen

Also fix a few coding style nits.

Fix misuse of Readonly in make-c-data

This script was initializing variables with Readonly via assignment
rather than the fat arrow.  The latter is the correct syntax.

Add option to bypass CrackLib for longer passwords

Add a trapdoor length, after which cracklib doesn't function.  This is
done via a cracklib_maxlen option to krb5.conf. passwords of that length
or shorter are still vetted.  (0, the default, performs the cracklib
fascist check at any length.)

CrackLib was designed to work with passwords, when passwords were 5-8
characters long.  CrackLib makes working with passphrases difficult,
rejecting some good ones.  The SQLite dictionary can be seeded with
Engligh trigrams, providing a compensating control.  Ergo, CrackLib
shouldn't be vetting passphrases; we should leave that to SQLite.

Add a NEWS entry for CrackLib security issues

Remove remaking CrackLib FascistGecos reference

All this code was previously removed, but there was still a
reference protected by #ifdef.  Get rid of that too.

Double buffer size in Mangle to allow for duplicate rules

No duplicating rules are enabled for the default rule set that is
run by the krb5-strength package, but close the latent security
vulnerability anyway.

Patch from Howard Guo <hguo@suse.com>.

Fix heimdal-history with the default DB_File::Lock

Change the DB_File::Lock calling method in heimdal-history to work
properly with the (buggy) CPAN version of DB_File::Lock, instead of
relying on Debian's patched version.  Thanks to Bernt Jernberg for the
report.

Prefer *.tar.xz in debian/watch to match packaging

Fix Upstream-Contact email address in debian/copyright

Begin error messages with a capital letter

Change the error messages returned for passwords that fail strength
checking to start with a capital letter.  This appears to be more
consistent with the error message conventions used inside Heimdal.

Check before removing /var/lib/heimdal-history

Add dependency on adduser

Manage a _history user and the history database directory

* Create a _history user and group and a /var/lib/heimdal-history
  directory on package installation for the use of heimdal-history,
  remove the user and the standard database on purge, and remove the
  directory if empty on package purge or removal.

Finalize changes for 3.0-1

Add pkg-config build dependency

Fix dependency for Perl DBD::SQLite package

Add upstream signing key and check upstream signatures

* Add the upstream signing key to debian/upstream/signing-key.asc and
  configure uscan to do signature validation.  Configure uscan to
  download the xz tarball instead of the gz tarball.

Add dependencies and description for new features

Also add an explanation for the Recommends so that people know
why they may need to install them.

Remove single-debian-patch local options

If I ever have to carry any patches, I'll use gbp pq, so remove
these settings.

Add changelog for upstream 3.0 release

Merge tag 'upstream/3.0' into debian

Upstream version 3.0

Imported Upstream version 3.0

Document minimum_different in heimdal-strength man page

Further README changes for the addition of SQLite and history

Release 3.0

Remove duplicate copyright declaration from heimdal-history

This confuses build-license

Add license statement to autogen

It's now long enough that build-license cares about it.

Skip more tests for valgrind checking

Also fully-qualify the path to the suppression file, which may
matter if the tests change working directories.

Fix a memory leak during SQLite dictionary initialization

Fix skipped test count with Heimdal and no SQLite

Stick to SQLite interfaces in 3.7

Use sqlite3_close instead of sqlite3_close_v2.  We don't need the
new semantics of sqlite3_close_v2.  Also use sqlite3_errmsg instead
of sqlite3_errstr, which actually appears to be better, assuming it
works as documented.

Replace remaining references to cdbmake-wordlist

Add additional Perl module requirements for bootstrap

Update to rra-c-util 5.4

* Use Lancaster Consensus environment variables to control tests.
* Work around perltidy bug that leaves behind stray log files.

RELEASE_TESTING, AUTOMATED_TESTING, and AUTHOR_TESTING are now used
to control whether to run tests that may not be of interest to the
end user.

Performance tuning of krb5-strength-wordlist for SQLite

Apply some performance tuning from various threads at Perl Monks
to increase the speed of the bulk creation of the SQLite database.

Fix typo in the wordlist-sqlite-t test

Refactor krb5-strength-wordlist

Separate the filter construction into a separate function and use
a hash for command-line arguments to make perlcritic happier with
the complexity of the main routine.

Fix perltidy error in wordlist-sqlite test

Add test for krb5-strength-wordlist SQLite generation

Separate krb5-strength-wordlist filtering to another test

Rather than merging the wordlist filtering test with the CDB test,
move it to a different unit test program.  This is probably overkill
for the tiny test that we do, but oh well.  It will make adding more
tests later somewhat easier if we ever do.

Add stopword for heimdal-strength documentation

Document the SQLite requirements in README

Add real Autoconf probing for SQLite

Change all the defines to look for HAVE_SQLITE instead of
HAVE_SQLITE3, since non-v3 versions are so old that I'm not going
to worry about the naming.  Add an Autoconf probe for SQLite that
tries pkg-config first and falls back on library probing.

Add a full complement of edit distance one SQLite checks

Fix one logic error uncovered by the more complete tests, which
produced the wrong result when the edit involved a sequence of
repeated characters.

Add support for SQLite dictionaries

The krb5-strength plugin and heimdal-strength program now support a
SQLite password dictionary.  This format of dictionary can detect any
password within edit distance one of a dictionary word, meaning that
the dictionary word can be formed by adding, removing, or changing a
single character in the password.  A SQLite password dictionary can be
used alone or in combination with any of the other supported
dictionary types.  SQLite dictionary support is based on work by David
Mazières.

Fix minor perlcritic errors in krb5-strength-wordlist

Add a missing stopword for spell checking and ignore the literal
newlines in the SQL code constants.

Fix CDB krb5-strength-wordlist check for no overwrite behavior

We now decline to overwrite an existing file, so adjust the test
suite to remove the generated file between tests.

Rename cdbmake-wordlist and add SQLite support

cdbmake-wordlist has been renamed to krb5-strength-wordlist.
Generating CDB dictionaries now requires the -c option; see the
documentation for more information.  A SQLite database of dictionary
words can now be created instead, using the -s option.

Wipe password copies before freeing them

We make a copy of the user's password in several places when doing
checks for passwords based on the user's principal.  Be sure to
wipe those copies with memset before freeing them.

Update to rra-c-util 5.3 and C TAP Harness 3.0

Update to rra-c-util 5.3:

* Fix portable/krb5.h build with a C++ compiler.

Update to C TAP Harness 3.0:

* Reopen standard input to /dev/null when running a test list.
* Don't leak extraneous file descriptors to tests.

Add a test suite for heimdal-history

Fix POD stopwords for heimdal-history

Fix the -s flag to heimdal-history (alternate quality checker)

heimdal-history claimed to support an -s option that specified a
different path to the quality check program to run, but it didn't
actually work.  Add support for it.

Support -q option to heimdal-history to suppress syslog

The -q option to heimdal-history will prevent it from logging to
syslog.  This is needed for the test suite to avoid spamming the
local syslog log.

Skip Perl strictness testing if module prereqs are missing

Modify the standard Perl strictness test, which also checks Perl
scripts for syntax errors, to support a list of prerequisite
modules.  Skip the test if any of those modules can't be loaded,
since they'll otherwise cause failures.  This makes the testing
more robust given that we have some scripts that require a bunch
of Perl modules not needed by the main package.

Add minimum_different configuration option

A new configuration option, minimum_different, can be set to require
that passwords contain at least that many unique characters.  This can
be used to reject long strings of identical characters or short
patterns, which may pass other checks but still be too easy to guess.