Imported Upstream version 2.1

Release 2.1

Add NEWS entries for TinyCDB build fixes

Update some of the password rejection error messages

Refer to "list of common passwords" when rejecting passwords due
to presence in a CDB dictionary, and say that passwords based on
the principal are based on "username or principal" to be more
technically accurate.

Simplify TinyCDB Autoconf probes

Refactor common code and ensure that rra_use_CDB is always set
properly.

Fix compilation without TinyCDB

The build without TinyCDB support was apparently not retested
after some refactoring, so some functions had the wrong signatures
or were not properly prototyped.

Fix handling of --with-tinycdb

Avoid adding -Lyes/lib and -Iyes/include to flags when given
--with-tinydb.  All this should do is make building with TinyCDB
mandatory.

Flesh out test suite requirements in README

Update BLURB in README based on the new Debian description

The previous version was rather outdated and didn't talk about
all of the plugin's capabilities.

Imported Upstream version 2.0

Fix distribution contents for the release

It's been a while and a lot of changes, so there were various
places where the contents of the distribution as defined by
Makefile.am were out of date.

Remove now-unneeded strlcat and strlcpy .gitignore entries

These tests are no longer included.

Fix logic to find the test CDB database

This is in the source package, not built dynamically as part of
the test suite, so we need to use test_file_path to find it.

Adjust the test suite for being run with a weird umask

If the test suite is run with a read-only source distribution, we
create files that aren't writable and then various things go awry.
Force permissions in a few key places to correct the problem.

Release 2.0

Rewrite LICENSE in Debian copyright-format 1.0

Fix copyright dates and cracklib/LICENCE reference in README

Remove a cdbmake-wordlist comment that confused build-license

One copyright statemnet per file works better.

Add a license statement to the heimdal-strength documentation

Add more standard sections to README

Add SUPPORT, SOURCE REPOSITORY, and LICENSE sections.

Fix a clang analysis error in the CDB code

The code was actually safe, but it wasn't clear enough for clang.
Add an explicit initialization to make the code clearer.

Add additional checks for passwords based on principals

The check for passwords based on the principal now check for passwords
formed by reversing or adding numbers before and after each separate
component of the principal.  This will catch passwords based on the
realm or components of the realm, which will often catch passwords
based on the name of the local institution.

Rename generic tests to principal tests

They're actually tests for passwords based on a principal, so name
them accordingly.

Document that checking against the principal is always done

Even if there are no dictionaries configured, we still check whether
the password is based on the principal.

Change strength_check to take a saner argument order

It makes more sense to pass in the principal before the password.
Change all the internal APIs to use that order.

Refactor checking for passwords based on principals

Move this code into a separate file in preparation for expanding
the nature of the checks, and following the general principal of
putting each type of check in a separate file.

Clean up and refactor configuration handling

Each "module" (CDB and CrackLib) now handles its own configuration
and setup, and the internal APIs are more straightforward and
simpler.

The plugin can now be configured without a dictionary, in which case
only the simpler checks available through the new configuration
variables are done.  This mode is mostly useful for testing, since
such simple checking can more easily be done via less complex password
strength configurations.

Use krb5_xfree for krb5_free_default_realm on Heimdal

The documentation says to use free, but this doesn't make sense
since the memory is allocated inside the Kerberos library.

Separate Kerberos configuration handling into a separate file

Provide a cleaner interface and hard-code the section values to
reduce the number of required arguments.  Factor out the realm
handling and properly free the default realms, avoiding memory
leaks.

Remove unnecessary includes in plugin/cracklib.c

Refactor the CrackLib code into a separate source file

Also rename api.c to general.c, which more accurately reflects the
current contents of that file.

Move CDB initialization into plugin/cdb.c

Move a bit farther down the path of having the different portions
of strength checking properly separated out.

Remove duplicate definition of KADM5_PASS_Q_GENERIC

This is now handled by portable/kadmin.h.

Refactor error handling inside the plugin

Add a set of generic functions for setting the error message in
the Kerberos context, move some of the error message strings out
where they can be easily manipulated, and use a similar error
message for CDB matches as the other password error messages.

Clean up error handling in the Heimdal plugin

Move the code to convert the Kerberos error to an error string
into a separate function and use that to simplify the error
reporting.  Remove an unnecessary prefix to the error for
initializing the password strength checking.

Rename the internal plugin functions

Use a standard prefix of strength_ rather than pwcheck_.  This
matches the name of the plugin and is somewhat more informative.

Change the default plugin install path and name

The default installation path for this plugin is now
/usr/local/lib/krb5/plugins/pwqual/strength.so (for both MIT and
Heimdal), assuming a --libdir setting of /usr/local/lib.  This may
require updates to the Kerberos KDC configuration or moving the plugin
when upgrading from earlier versions.

Fix Heimdal pwcheck header probing

The configure probe was never going to work properly.  Fix that by
adding the correct includes.  Drop the fallback for versions of
Heimdal without the include file installed, which is not required
for Debian squeeze and later.

Hide internal functions

Mark all the internal plugin functions and all the CrackLib functions
as hidden.

Mark the portable_dummy symbol as hidden

This was leaking out of shared libraries that used libportable.
Whoops.

Remove strlcat and strlcpy portable replacements

I no longer use either function in this package, so there's no
need to include the replacements and the code to test them.

Reorganize the plugin tests

Make the organization of the test suite reflect the organization
of the source code.

Update EXTRA_DIST for file changes

Various things have been moved around and a bunch of new files
added.  Update EXTRA_DIST accordingly.

Move heimdal-strength test

Now that the program is in tools, move the test accordingly.

Rename plugin/api.h to plugin/internal.h

This no longer defines the default API, since that is now an
internal implementation detail.  Rename the header file to match
the sort of thing I do with other projects.

Use krb5_free_string to free krb5_appdefault_string results

After parsing a numeric setting, free the string returned by
krb5_appdefault_string with krb5_free_string instead of free.

Add configurable checks for ASCII and for non-letters

New boolean settings require_ascii_printable and require_non_letter
are supported in the krb5-strength setting of [appdefaults] in
krb5.conf.  The former rejects passwords containing characters other
than printable ASCII characters (including space), and the latter
requires that passwords contain at least one character that is not a
letter (upper or lower case) or a space.

Support UTF-8 data in password test cases

In make-c-data, when generating C source from the JSON data,
properly encode UTF-8 strings.  This still requires a C compiler
that can handle raw UTF-8 input, but hopefully that won't be a
serious portability issue.

Add support for enforcing a minimum password length

This is configured using the minimum_length setting in krb5.conf
in the krb5-strength section of [appdefaults].  This length check
is independent of CrackLib's checks.

Probe for setrlimit for the xmalloc tests

Testing the xmalloc utility library requires knowing whether
setrlimit exists.  Add the check to configure.

Separate generic tests from CrackLib tests

Move the generic password tests into a separate file so that
they can be run independent from the CrackLib tests, and run
them for both CrackLib and CDB configurations.

Move password test data into a subdirectory

The number of files is only going to grow, so move it into a
subdirectory and use a loop in autogen to generate C versions of
every file to make it easier to add new blocks of test data.

Add Perl testing from rra-c-util

Delete the old, simple POD tests that only looked at the docs for
heimdal-strength and add full POD and Perl testing using the
generic tests from rra-c-util.

Add missing POD spelling stopword for cdbmake-wordlist

Fix style problems in the heimdal/external test

Some style and formatting issues crept in via various rewrites.
Clean those up again.

Fix naming of cdbmake-wordlist test

We've been using -t instead of .t since this is an Automake
project.  Be consistent.

Add test suite for util/messages-krb5

Move heimdal-strength to the tools directory

Try to keep the proliferation of directories to a minimum and put
all the command-line tools in the same directory.

Renamed cdb.m4 to tinycdb.m4

Reflect the contents more accurately in the name.

Add support for checking against a CDB database

Add support for building with TinyCDB and then checking passwords
against a CDB database.  There is a new password_dictionary_cdb
krb5.conf configuration setting that configures a CDB directory to
use.  The tests with a CDB dictionary are much simpler: passwords are
rejected if found in the dictionary either literally, with one or two
characters removed from the start or end, or with one character
removed from both the start and the end.  Both a CrackLib and a CDB
dictionary can be specified to check both dictionaries.

Include strerror results in Kerberos error messages

When reporting a Kerberos error message from a system call failure,
include the strerror results in the message.

Add cdbmake-wordlist test to the default test suite

Add a test suite for cdbmake-wordlist

Add initial version of cdbmake-wordlist tool

This creates a *.cdb file from a wordlist and will be used for the
new cdb support coming to krb5-strength.  For right now, there's no
test suite.

Remove appdefaults realm support for MIT from TODO

This was completed.

Use the Kerberos context to set the error message

The plugin now sets the Kerberos error message in the context to pass
error information, resulting in higher-quality error reporting in the
MIT Kerberos plugin.

Use die_krb5 in the external password checker

This is pointless at the moment, since we're only using it for
failures in krb5_init_context, but it will be more useful later.

Move krb5.conf configuration into the plugin

Load the configuration inside the plugin when we initialize it,
and pass in a Kerberos context to the plugin so that this is
possible.  Obtain or pass in an existing Kerberos context in the
places where we weren't already doing so.

This loses some more detailed error reporting, particularly around
non-existent configured dictionaries, which will be restored later
by using the Kerberos error message.

Fix memory management in the MIT plugin test

We were freeing one of our testing paths before using it again.

Remove some reachable leaks, static data in cracklib, tests

Don't attempt an optimization where we keep the last block of
the dictionary in memory to answer questions, since each call
may use a different dictionary.  Close the password dictionary
after each lookup (although this doesn't recover all of the
memory due to more static data elsewhere).

Free more memory at the end of the MIT plugin test.

Use standard CrackLib tools when not using embedded

When building with the system CrackLib, use the standard tools to
build the test dictionary instead of building the embedded code
just to use the packer.

Also use the tools to build the dictionary so that the word list
in the test suite doesn't have to be sorted.

Remove explicit CPPFLAGS setting for the plugin

This is handled by the global setting.

Update EXTRA_DIST and libportable sources

Add the new header files to libportable so that they're included
in the distribution, and remove obsolete files from EXTRA_DIST.

Make the way too short error consistent with CrackLib

Current CrackLib returns "it is WAY too short" rather than
"it's WAY too short".  Be consistent so that more of the test
suite will pass correctly.

Rewrite the MIT plugin tests in C

Rewrite the MIT plugin test in pure C, using the generated C
data and calling the shell script to create the necessary krb5.conf
file.  Check configuration both via the internal dictionary path
and via appdefaults.

Fix configuration and status codes in the MIT plugin

Modify the MIT plugin to support [appdefaults] configuration in
addition to the dictionary path configured in the KDC.  Modify the
core code to return meaningful error codes instead of just 0 and 1,
adjust the Heimdal module accordingly to translate them into 0 and
1, and use that to allow the MIT code to return real error codes.

In order to support this, import more of the Kerberos portability
layer including new portability for kadmin headers.

Report a real plan for the heimdal/external test

We can calculate how many tests we're going to run, so do so and
report a real plan.

Rewrite the Heimdal plugin tests in C

Add a new Perl program to generate a C version of the password
test data from the JSON source, and run that program during
autogen so that we can ship the source in the distribution and
not require JSON support for the basic test suite.  Add a new
shell script to generate the necessary krb5.conf file.

Rewrite the Heimdal plugin test in pure C, using the generated C
data and calling the shell script to create the necessary krb5.conf
file.

Rewrite the heimdal-strength tests in Perl

Externalize the passwords and expected results in a JSON file
and rewrite the test case in Perl, making it read the list of
test cases from that file.  Add a stripped-down krb5.conf file
used by the test suite instead of trying to manipulate the system
krb5.conf file.

Import Perl test support modules from rra-c-util

Import Test::RRA, Test::RRA::Automake, and Test::RRA::Config
from rra-c-util 4.10 for the use of test cases written in Perl.

Clean up the leading comment on the MIT plugin code

Reformat the prototype for pwqual_strength_initvt

Use util/macros.h for the UNUSED macro

Also reorganize the MIT plugin code a little bit.

Import the util layer from rra-c-util, use for heimdal-strength

Rather than rolling our own versions of xmalloc, xstrdup, and die,
import the util layer and use the standard versions of those
functions.  Also import the test suite.

Use the correct Stanford copyright notice, update dates

Include portable/krb5.h instead of krb5.h

Make use of the portability layer to figure out which Kerberos
header to include.

Remove an old TODO item related to the MIT patch

Don't purge the autom4te.cache directory

Retain this to make subsequent autogen runs faster.

Use the portability layer and close a memory leak

Free the unparsed principal name when checking passwords in
Heimdal, and use the portability layer in the external strength
program.

Add a more extensive Kerberos portability layer

We don't really care about portability for krb5_get_error_message
since we effectively will require that it exists, but may as well
use the same structure as other code.  Add some other functions
that we're using (or should be using).

Add support for new MIT plugin interface, drop old patch

Add support for the MIT Kerberos password quality plugin interface,
available in MIT Kerberos 1.9 and later, contributed by Greg Hudson
and MIT.  Drop the patch for MIT Kerberos 1.4 (and hence support for
versions of MIT Kerberos prior to 1.9).

Add additional CrackLib changes to cracklib/HISTORY

Fix the path to the Heimdal password strength header

Expect the Heimdal password strength checking plugin header in
kadm5/kadm5-pwcheck.h instead of outside of the kadm5 directory.  This
is the path used by current versions of Heimdal.

Update to the latest warning flags fromr rra-c-util 4.9

Add a missing ANSI C prototyping fix

Remove the need for tests/data/.placeholder

Just create the tests/data directory if needed before running
the packer to create the dictionary.

Update to rra-c-util 4.9 and C TAP Harness 2.2

Update to rra-c-util 4.9:

* Probe for Kerberos headers using file checks instead of compiles.
* Improve probe for the Heimdal libroken library.
* Always build with large file support.
* Conditionally call AM_PROG_AR for portability to new Autotools.

Update to C TAP Harness 2.2:

* Allow more easily running single programs under tests/runtests.
* Flush the output from the test harness after each test.

Make the simplicity check dependent on password length

CrackLib checks for passwords where a character is a simple increment
or decrement of the previous character.  In previous versions, the
embedded version of CrackLib allowed at most four such occurrences in
the entire password.  This results in false positives on long
passphrases, since such accidental letter relationships aren't
uncommon in human languages.  Change the embedded CrackLib to allow
one such simple increment for every three characters in the password,
which tightens the check somewhat for shorter passwords and loosens it
considerably for longer passwords.

Imported Upstream version 1.1

Add new rra-c-util files to the distribution