php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #48441 ldap_search() sizelimit, timelimit and deref options persist
Submitted: 2009-05-31 21:53 UTC Modified: 2009-06-15 15:24 UTC
From: patrickallaert@php.net Assigned: patrickallaert (profile)
Status: Closed Package: LDAP related
PHP Version: 5.*, 6CVS (2009-06-01) OS: *
Private report: No CVE-ID: None
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: patrickallaert@php.net
New email:
PHP Version: OS:

 

 [2009-05-31 21:53 UTC] patrickallaert@php.net 
Description:
------------
Specifiying sizelimit, timelimit and deref options in ldap_search() makes them persistent for next call to ldap_search().
<?php
// ldap_search() with sizelimit, timelimit and deref options
ldap_search($link, $dn, $filter, $attrs, 1, 1, 1, LDAP_DEREF_ALWAYS);

// Default value not used for sizelimit, timelimit and deref, previous one used.
// Returns same truncated (due to sizelimit) results as previous call
ldap_search($link, $dn, $filter, $attrs, 1);
?>

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2009-06-01 09:02 UTC] jani@php.net 
How about not. These kind of "magic" things tend to cause bugs that are 
next to impossible to find.
 [2009-06-01 10:24 UTC] patrickallaert@php.net 
I am not sure to understand your remark.
What I described here is the *current* way ldap_search() works.
If I'd like to set those options for all queries on the same link, I'd rather use ldap_set_option() than side effects of ldap_search().
I fixed it locally and I am about to post my patch on internals for review before commiting it to CVS (as I am rather new @php.net)
 [2009-06-02 10:59 UTC] jani@php.net 
Reopened. I misunderstood your report. :)
Note: bug fixes should go also to PHP_5_2.
 [2009-06-15 15:24 UTC] patrickallaert@php.net 
This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Nov 21 13:01:29 2024 UTC