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(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel