[PATCH] gensec: Change log level of message when no PAC is found

Jeremy Allison jra at samba.org
Fri Apr 29 22:57:57 UTC 2016


On Fri, Apr 29, 2016 at 01:24:03PM -0700, Christof Schmitt wrote:
> From ee44746c82e33e9596a30333c33e36edb7058c28 Mon Sep 17 00:00:00 2001
> From: Christof Schmitt <cs at samba.org>
> Date: Thu, 28 Apr 2016 11:23:41 -0700
> Subject: [PATCH] gensec: Change log level of message when no PAC is found
> 
> For a Samba server that uses a non-AD KDC this message is triggered on
> every new connection. Change the log level from warning/1 to a more
> appropriate notice/5.
> 
> Signed-off-by: Christof Schmitt <cs at samba.org>

LGTM. Pushed !

>  auth/gensec/gensec_util.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/auth/gensec/gensec_util.c b/auth/gensec/gensec_util.c
> index 64fffb1..da2e2e5 100644
> --- a/auth/gensec/gensec_util.c
> +++ b/auth/gensec/gensec_util.c
> @@ -48,8 +48,8 @@ NTSTATUS gensec_generate_session_info_pac(TALLOC_CTX *mem_ctx,
>  				  principal_string));
>  			return NT_STATUS_ACCESS_DENIED;
>  		}
> -		DEBUG(1, ("Unable to find PAC for %s, resorting to local user lookup\n",
> -			  principal_string));
> +		DBG_NOTICE("Unable to find PAC for %s, resorting to local "
> +			   "user lookup\n", principal_string);
>  	}
>  
>  	if (gensec_security->auth_context && gensec_security->auth_context->generate_session_info_pac) {
> -- 
> 1.8.3.1
> 



More information about the samba-technical mailing list