[PATCH] Improve logging for localauth MIT Kerberos plugin
asn at samba.org
Mon Feb 18 10:02:16 UTC 2019
On Monday, February 18, 2019 10:29:04 AM CET Noel Power via samba-technical
> Hi Andreas
> 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 Schneider asn at samba.org
Samba Team www.samba.org
More information about the samba-technical