From: Aki Tuomi <aki.tuomi@dovecot.fi>
Date: Thu, 27 Apr 2017 08:53:13 +0000 (+0300)
Subject: auth: Use MECH subsystem when logging error about skipping all password databases
X-Git-Tag: 2.3.0.rc1~1682
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2b7fbe3badf96da1491293db7322da42cc27fad9;p=thirdparty%2Fdovecot%2Fcore.git

auth: Use MECH subsystem when logging error about skipping all password databases

Otherwise it will assert-crash because all password databases were skipped.
---

diff --git a/src/auth/auth-request.c b/src/auth/auth-request.c
index 53dc237be8..7f11d4fc2c 100644
--- a/src/auth/auth-request.c
+++ b/src/auth/auth-request.c
@@ -1027,10 +1027,8 @@ void auth_request_verify_plain_continue(struct auth_request *request,
 	request->passdb = passdb;
 
 	if (passdb == NULL) {
-		auth_request_log_error(request, AUTH_SUBSYS_DB,
-			"All password databases were skipped for mechanism '%s'",
-				request->mech == NULL ? "<empty>"
-						      : request->mech->mech_name);
+		auth_request_log_error(request, AUTH_SUBSYS_MECH,
+			"All password databases were skipped");
 		callback(PASSDB_RESULT_INTERNAL_FAILURE, request);
 		return;
 	}
@@ -1208,10 +1206,8 @@ void auth_request_lookup_credentials_policy_continue(struct auth_request *reques
 	request->passdb = passdb;
 
 	if (passdb == NULL) {
-		auth_request_log_error(request, AUTH_SUBSYS_DB,
-			"All password databases were skipped for mechanism '%s'",
-				request->mech == NULL ? "<empty>"
-						      : request->mech->mech_name);
+		auth_request_log_error(request, AUTH_SUBSYS_MECH,
+			"All password databases were skipped");
 		callback(PASSDB_RESULT_INTERNAL_FAILURE, NULL, 0, request);
 		return;
 	}