[SSSD] [PATCHES] CONFIGURE: Enhance detection of pam

Sumit Bose sbose at redhat.com
Tue Jun 3 07:06:53 UTC 2014


On Tue, Jun 03, 2014 at 08:15:36AM +0200, Lukas Slebodnik wrote:
> On (02/06/14 23:19), Sumit Bose wrote:
> >On Sat, May 31, 2014 at 04:59:44PM +0200, Lukas Slebodnik wrote:
> >> ehlo,
> >> 
> >> I think that the attached patches are self descriptive.
> >> 
> >> LS
> >
> >Patches apply and build fine. I've run build on a few platforms without
> >any issue, but I had none with openpam at hand.
> >
> >I only have one comment on the last patch. getlogin() is not thread safe
> It is
> https://github.com/freebsd/freebsd/blob/master/lib/libc/gen/getlogin.c
>     lines 72:82

There are locks, but the result is still in a static variable (line 59),
i.e. in the admittedly rare case when two threads do authentication in
parallel for different users one thread might override the result of the
other.

bye,
Sumit

> 
> LS
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel



More information about the sssd-devel mailing list