[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: slapd: errno and Debug() function (ITS#276)



I committed a fix to OPENLDAP_REL_ENG_1_2.  Please test.

Kurt

At 09:23 PM 8/31/99 GMT, you wrote:
>Full_Name: J.R. Heisey
>Version: 1.2
>OS: Windows NT
>URL: ftp://ftp.openldap.org/incoming/
>Submission from: (NULL) (209.157.115.69)
>
>
>Submit an ITS to remind me to port the -devel fix to 1.2.
>        Kurt
>
>At 11:48 AM 8/31/99 -0700, you wrote:
>>In send_search_entry() errno is tested to determine what
>>error occured in ber_flush(). However two other functions
>>are called before errno is tested those being Debug()
>>and ldap_pvt_thread_mutex_unlock(). I did look into the
>>thread function but Debug definitely has the potential
>>of changing the value of errno before the test.
>>
>>This will get more complicated for a Windows NT port
>>because WSAGetLastError() needs to be called to test
>>for EWOULDBLOCK instead of testing for errno.
>>
>>Same issue in send_ldap_result2().
>>
>>Perhaps Debug() should store and reset the errno value.
>
>
>