What I meant was, make sure that ldap_get_values_len() is also present
in the LDAP libraries on Solaris and Windows.
I've checked the docs on those platforms, and they both have those APIs.
The reason I bring it up is because those 3 implementations have 98%
the same API but some differences in so
Wez Furlong wrote:
> My really really late 2 cents on this.
>
> Please make sure that your changes don't make the extension depend on
> OpenLDAP. Solaris and Windows LDAP implementations are close but not
> totally the same as OpenLDAP.
>
> I haven't looked at your patches and probably won't hav
My really really late 2 cents on this.
Please make sure that your changes don't make the extension depend on
OpenLDAP. Solaris and Windows LDAP implementations are close but not
totally the same as OpenLDAP.
I haven't looked at your patches and probably won't have time to do
so; I'm merely doin
Antony Dovgal wrote:
> On 03/01/2007 01:11 AM, Doug Goldstein wrote:
>> Antony Dovgal wrote:
>>> On 03/01/2007 12:35 AM, Doug Goldstein wrote:
> Did you really test it with non-NULL terminated strings?
> Don't you need to add '\0' manually?
The test is that you run the example cod
On 03/01/2007 01:11 AM, Doug Goldstein wrote:
Antony Dovgal wrote:
On 03/01/2007 12:35 AM, Doug Goldstein wrote:
Did you really test it with non-NULL terminated strings?
Don't you need to add '\0' manually?
The test is that you run the example code from bug #38819, watch PHP
crash. Apply my p
Antony Dovgal wrote:
> On 03/01/2007 12:35 AM, Doug Goldstein wrote:
>>> Did you really test it with non-NULL terminated strings?
>>> Don't you need to add '\0' manually?
>>
>> The test is that you run the example code from bug #38819, watch PHP
>> crash. Apply my patch and watch PHP not crash. Fai
On 03/01/2007 12:46 AM, Antony Dovgal wrote:
I removed PHP_FE(ldap_get_values) because it's a pointless function.
The point is that the patch should FIX this function, not REMOVE it.
And it's still there, but commented out (which makes not sense).
Uhm.. Ignore that, I need to sleep more.
--
On 03/01/2007 12:35 AM, Doug Goldstein wrote:
Did you really test it with non-NULL terminated strings?
Don't you need to add '\0' manually?
The test is that you run the example code from bug #38819, watch PHP
crash. Apply my patch and watch PHP not crash. Fairly simple. My backtrace
is identica
Antony Dovgal wrote:
> On 03/01/2007 12:04 AM, Doug Goldstein wrote:
>>> Wait, I thought the DEPRECATED thing fixed it (I can't test it myself
>>> as I
>>> don't use LDAP).
>>> If not, then what was it all about?
>>
>> The test case still failed for me. It also does not address the fact
>> that
>>
On 03/01/2007 12:04 AM, Doug Goldstein wrote:
Wait, I thought the DEPRECATED thing fixed it (I can't test it myself as I
don't use LDAP).
If not, then what was it all about?
The test case still failed for me. It also does not address the fact that
the usage of the PHP ldap functions is unsafe b
Antony Dovgal wrote:
> Hello.
> Thanks for sending the patch here.
>
> On 02/28/2007 11:33 PM, Doug Goldstein wrote:
>> Referencing Bug #38819 & Bug #40671
>> http://bugs.php.net/bug.php?id=38819
>> http://bugs.php.net/bug.php?id=40671
>>
>> Essentially I looked through the above mentioned bug, the
Hello.
Thanks for sending the patch here.
On 02/28/2007 11:33 PM, Doug Goldstein wrote:
Referencing Bug #38819 & Bug #40671
http://bugs.php.net/bug.php?id=38819
http://bugs.php.net/bug.php?id=40671
Essentially I looked through the above mentioned bug, the bugs opened
with OpenLDAP developers, a
12 matches
Mail list logo
