[PATCH] Fix a memory leak in pam_smbpass

Volker Lendecke Volker.Lendecke at SerNet.DE
Thu Jan 22 04:14:15 MST 2015


On Thu, Jan 22, 2015 at 10:45:06AM +0100, Andreas Schneider wrote:
> Hello,
> 
> attached is a patch which correctly frees memory allocated by 
> pam_sm_authenticate().
> 
> Review and push appreciated!

R-b: Me, although free() deals fine with NULL, so the if-statement in
ret_data_cleanup isn't necessary.

This triggers one question: Is pam_smbpass really going to be maintained
in the future? Isn't pam_winbind with winbind running locally much better?

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de


More information about the samba-technical mailing list