[PATCH] Improve logging for localauth MIT Kerberos plugin

[Andreas Schneider]

On Monday, February 18, 2019 10:29:04 AM CET Noel Power via samba-technical 
wrote:
> Hi Andreas

Hi Noel,
 
> On 15/02/2019 09:52, Andreas Schneider via samba-technical wrote:
> > Hello,
> > 
> > attached is a patch to improve the logging for the MIT KRB5 localauth
> > plugin.
> > 
> > 
> > Review is much appreciated.
> > 
> > 
> > Thanks,
> > 
> > 	Andreas
> 
> initially I thought this was maybe the wrong patch attached (or wrong
> commit message) I saw was the reorganization to fix the leaked principle
> (but took me a while to see the log part). Not sure if the leak fix
> deserves a separate patch or just the commit message needs to mention
> it. other than this lgtm RB


Thanks for looking into that.


I don't see a leak fix in that patch. As the princ_str is user later in the 
log message too, we need to keep it around till we printed it and then free 
it. So the call for freeing it just got moved to the end. So it made sense to 
use a "destructor" for freeing it.


	Andreas



-- 
Andreas Schneider                      asn at samba.org
Samba Team                             www.samba.org
GPG-ID:     8DFF53E18F2ABC8D8F3C92237EE0FC4DCC014E3D