|
 |
963210 |
From 9c0a06f38189d255575acdae5efb22b76b4c33b3 Mon Sep 17 00:00:00 2001
|
|
|
963210 |
From: Robbie Harwood <rharwood@redhat.com>
|
|
|
963210 |
Date: Mon, 13 Nov 2017 13:32:37 -0500
|
|
|
963210 |
Subject: [PATCH] Expose context errors in pkinit_server_plugin_init
|
|
|
963210 |
|
|
|
963210 |
Commit 3ff426b9048a8024e5c175256c63cd0ad0572320 attempted to display
|
|
|
963210 |
an error when OCSP support was requested, but this error message was
|
|
|
963210 |
suppressed in pkinit_server_plugin_init(). Add a trace log for each
|
|
|
963210 |
realm initialization error, and pass through the realm initialization
|
|
|
963210 |
error when the KDC serves only one realm. Other error messages from
|
|
|
963210 |
pkinit_init_kdc_profile(), such as missing pkinit_identity or
|
|
|
963210 |
pkinit_anchors, are also now exposted.
|
|
|
963210 |
|
|
|
963210 |
[ghudson@mit.edu: clarified commit message]
|
|
|
963210 |
|
|
|
963210 |
ticket: 8621 (new)
|
|
|
963210 |
target_version: 1.16
|
|
|
963210 |
tags: pullup
|
|
|
963210 |
|
|
|
963210 |
(cherry picked from commit 225aab3540c13c6289b22022d5e110f6fc26151d)
|
|
|
963210 |
---
|
|
|
963210 |
src/plugins/preauth/pkinit/pkinit_srv.c | 19 +++++++++++++------
|
|
|
963210 |
src/plugins/preauth/pkinit/pkinit_trace.h | 3 +++
|
|
|
963210 |
2 files changed, 16 insertions(+), 6 deletions(-)
|
|
|
963210 |
|
|
|
963210 |
diff --git a/src/plugins/preauth/pkinit/pkinit_srv.c b/src/plugins/preauth/pkinit/pkinit_srv.c
|
|
|
963210 |
index 8e77606f8..143d331a2 100644
|
|
|
963210 |
--- a/src/plugins/preauth/pkinit/pkinit_srv.c
|
|
|
963210 |
+++ b/src/plugins/preauth/pkinit/pkinit_srv.c
|
|
|
963210 |
@@ -1622,16 +1622,23 @@ pkinit_server_plugin_init(krb5_context context,
|
|
|
963210 |
|
|
|
963210 |
for (i = 0, j = 0; i < numrealms; i++) {
|
|
|
963210 |
TRACE_PKINIT_SERVER_INIT_REALM(context, realmnames[i]);
|
|
|
963210 |
- retval = pkinit_server_plugin_init_realm(context, realmnames[i], &plgctx);
|
|
|
963210 |
- if (retval == 0 && plgctx != NULL)
|
|
|
963210 |
+ krb5_clear_error_message(context);
|
|
|
963210 |
+ retval = pkinit_server_plugin_init_realm(context, realmnames[i],
|
|
|
963210 |
+ &plgctx);
|
|
|
963210 |
+ if (retval)
|
|
|
963210 |
+ TRACE_PKINIT_SERVER_INIT_FAIL(context, realmnames[i], retval);
|
|
|
963210 |
+ else
|
|
|
963210 |
realm_contexts[j++] = plgctx;
|
|
|
963210 |
}
|
|
|
963210 |
|
|
|
963210 |
if (j == 0) {
|
|
|
963210 |
- retval = EINVAL;
|
|
|
963210 |
- krb5_set_error_message(context, retval,
|
|
|
963210 |
- _("No realms configured correctly for pkinit "
|
|
|
963210 |
- "support"));
|
|
|
963210 |
+ if (numrealms == 1) {
|
|
|
963210 |
+ k5_prependmsg(context, retval, "PKINIT initialization failed");
|
|
|
963210 |
+ } else {
|
|
|
963210 |
+ retval = EINVAL;
|
|
|
963210 |
+ k5_setmsg(context, retval,
|
|
|
963210 |
+ _("No realms configured correctly for pkinit support"));
|
|
|
963210 |
+ }
|
|
|
963210 |
goto errout;
|
|
|
963210 |
}
|
|
|
963210 |
|
|
|
963210 |
diff --git a/src/plugins/preauth/pkinit/pkinit_trace.h b/src/plugins/preauth/pkinit/pkinit_trace.h
|
|
|
963210 |
index 6abe28c0c..8d489469f 100644
|
|
|
963210 |
--- a/src/plugins/preauth/pkinit/pkinit_trace.h
|
|
|
963210 |
+++ b/src/plugins/preauth/pkinit/pkinit_trace.h
|
|
|
963210 |
@@ -100,6 +100,9 @@
|
|
|
963210 |
TRACE(c, "PKINIT server skipping EKU check due to configuration")
|
|
|
963210 |
#define TRACE_PKINIT_SERVER_INIT_REALM(c, realm) \
|
|
|
963210 |
TRACE(c, "PKINIT server initializing realm {str}", realm)
|
|
|
963210 |
+#define TRACE_PKINIT_SERVER_INIT_FAIL(c, realm, retval) \
|
|
|
963210 |
+ TRACE(c, "PKINIT server initialization failed for realm {str}: {kerr}", \
|
|
|
963210 |
+ realm, retval)
|
|
|
963210 |
#define TRACE_PKINIT_SERVER_MATCHING_UPN_FOUND(c) \
|
|
|
963210 |
TRACE(c, "PKINIT server found a matching UPN SAN in client cert")
|
|
|
963210 |
#define TRACE_PKINIT_SERVER_MATCHING_SAN_FOUND(c) \
|