Mercurial > dovecot > core-2.2

changeset 22021:06d37092bc08

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
auth: Do not use AUTH_SUBSYS_MECH in logging when request->mech is NULL This can happen when a lookup is done instead of authentication. Fixes auth: Panic: file auth-request.c: line 2560 (get_log_prefix): assertion failed: (auth_request->mech != NULL)
author Aki Tuomi <aki.tuomi@dovecot.fi>
date Tue, 09 May 2017 12:25:49 +0300
parents f08ffc29981c
children e552f75d51b2
files src/auth/auth-request.c
diffstat 1 files changed, 6 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/auth/auth-request.c	Mon May 08 12:24:52 2017 +0300
+++ b/src/auth/auth-request.c	Tue May 09 12:25:49 2017 +0300
@@ -674,14 +674,18 @@
 	username = request->user;
 
 	if (!auth_request_mechanism_accepted(mechs, request->mech)) {
-		auth_request_log_debug(request, AUTH_SUBSYS_MECH,
+		auth_request_log_debug(request,
+				       request->mech != NULL ? AUTH_SUBSYS_MECH
+							      : "none",
 				       "skipping passdb: mechanism filtered");
 		return TRUE;
 	}
 
 	if (passdb->passdb->username_filter != NULL &&
 	    !auth_request_username_accepted(username_filter, username)) {
-		auth_request_log_debug(request, AUTH_SUBSYS_MECH,
+		auth_request_log_debug(request,
+				       request->mech != NULL ? AUTH_SUBSYS_MECH
+							      : "none",
 				       "skipping passdb: username filtered");
 		return TRUE;
 	}