ID: 24084 Updated by: [EMAIL PROTECTED] Reported By: peter_c60 at hotmail dot com -Status: Open +Status: Closed Bug Type: LDAP related Operating System: Linux PHP Version: 4.3.2 New Comment:
Should be fixed now. (Your config.m4 patch was not very good, the ldap libs can be static too, so ldd wouldn't work very well. :) Previous Comments: ------------------------------------------------------------------------ [2003-06-09 18:42:04] peter_c60 at hotmail dot com I have managed to compile and test the new patch and it works AFAICT. ------------------------------------------------------------------------ [2003-06-09 13:22:04] peter_c60 at hotmail dot com It looks like the patch checked into CVS is wrong, at least from an autoconf point of view. The problem is that the function ldap_sasl_interactive_bind_s is always defined whether SASL was enabled at time of compilation or not. Also the sasl.h header is required because the interactive function requires some of its defines. I've made some new patches that check that the LDAP library was linked against libsasl(2) (using ldd, I'm not sure if this is the correct method on all platforms) and also checks for the headers. I haven't tested it myself because I keep on getting a libtool error at time of compile (but that's a story for another bug report) but it seems to work correctly up to the configure stage. Anyway here are the patches (to be applied to the current CVS version): http://www.geocities.com/ldappatch/config2.txt (apply to config.m4) http://www.geocities.com/ldappatch/ldap2.txt (apply to ldap.c) http://www.geocities.com/ldappatch/php_ldap2.txt (apply to php_ldap.h) ------------------------------------------------------------------------ [2003-06-08 18:44:26] [EMAIL PROTECTED] Patch committed to CVS. (in php5/) ------------------------------------------------------------------------ [2003-06-08 16:19:32] [EMAIL PROTECTED] The problem with this patch is that it never checks if SASL support is enabled in your LDAP library. I think you will need to check for this with config.m4 and add some ifdef's to the code accordingly, unless *every* LDAP library comes with SASL support of course. ------------------------------------------------------------------------ [2003-06-08 16:15:51] peter_c60 at hotmail dot com OK then: http://www.geocities.com/ldappatch/ldap.txt http://www.geocities.com/ldappatch/php_ldap.txt ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/24084 -- Edit this bug report at http://bugs.php.net/?id=24084&edit=1
