diff options
author | Russ Allbery <eagle@eyrie.org> | 2016-04-26 20:52:34 -0700 |
---|---|---|
committer | Russ Allbery <eagle@eyrie.org> | 2016-04-26 20:52:34 -0700 |
commit | 33e252deff8149506371578a0782c0b96ab43886 (patch) | |
tree | 475f48eac6b3c3f1c69aa39e6a8eee169c1e03a1 /tests | |
parent | db19c1d8b1de7d26aac07c722373391cef2ed7f1 (diff) |
Coding style and comment fixes for the last patch
Diffstat (limited to 'tests')
-rw-r--r-- | tests/server/anonymous-t.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/tests/server/anonymous-t.c b/tests/server/anonymous-t.c index e118ddd..32d1115 100644 --- a/tests/server/anonymous-t.c +++ b/tests/server/anonymous-t.c @@ -1,7 +1,7 @@ /* * Test suite for anonymous authentication. * - * Copyright 2015 Russ Allbery <eagle@eyrie.org> + * Copyright 2015, 2016 Russ Allbery <eagle@eyrie.org> * * See LICENSE for licensing terms. */ @@ -25,9 +25,16 @@ * that we can get a service ticket for the provided principal, and return the * name of the Kerberos ticket cache on success and NULL on failure. Internal * Kerberos errors resort in an abort instead. + * + * Some older versions of Heimdal not only can't do PKINIT, but also crash + * when krb5_get_init_creds_password is called with no password or prompter. + * Heimdal versions which avoid the crash have krb5_init_creds_set_password, + * as do all versions of MIT Kerberos which support PKINIT. So, disable the + * anonymous tests if that function is not present. */ #ifdef HAVE_KRB5 -# if !defined(HAVE_KRB5_GET_INIT_CREDS_OPT_SET_ANONYMOUS) || !defined(HAVE_KRB5_INIT_CREDS_SET_PASSWORD) +# if !defined(HAVE_KRB5_GET_INIT_CREDS_OPT_SET_ANONYMOUS) \ + || !defined(HAVE_KRB5_INIT_CREDS_SET_PASSWORD) static char * cache_init_anonymous(krb5_context ctx UNUSED, const char *principal UNUSED) |