On Fri, Apr 20, 2018 at 2:09 PM, Thomas Munro
<thomas.munro@enterprisedb.com> wrote:
> On Fri, Apr 20, 2018 at 1:43 PM, Greg k <gregg.kay@gmail.com> wrote:
>> Do I need to raise this as an issue or bug?
>
> It seems like we might need a patch on that adds ldap_set_option(ldap,
> LDAP_OPT_RESTART, LDAP_OPT_ON) somewhere in auth.c to tell it to
> restart on EINTR.
I'm a bit confused about that; how are you supposed to do that early
enough to cover all its syscalls if you don't have the handle until
after you've connected? Perhaps CheckLDAPAuth() (or higher up) should
be blocking signals instead, or at least SIGUSR1 (I guess that's what
you're receiving -- can you tell?)
--
Thomas Munro
http://www.enterprisedb.com