DO NOT REPLY [Bug 34655] New: - crypt_r check is broken for non-threaded case

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=34655>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=34655

           Summary: crypt_r check is broken for non-threaded case
           Product: APR
           Version: 0.9.6
          Platform: Other
        OS/Version: AIX
            Status: NEW
          Severity: normal
          Priority: P2
         Component: APR-util
        AssignedTo: bugs@httpd.apache.org
        ReportedBy: rmurray+apache@cyberhqz.com


apr.h gets created with #defined APR_HAS_THREADS 0 when building without thread
support.
apr-util/crypto/apr_md5.c:719 has the check:
#if defined(_AIX) && defined(APR_HAS_THREADS)
the latter part of this check is always true as APR_HAS_THREADS is defined,
whether it is true or false, which means you always have to have crypt_r, even
if you aren't building a threaded lib.  The fix I'm using is to remove the
defined() part of the second test.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org