[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