#33588 [Asn->Csd]: No RootDSE query possible
| From: | sniper@php.net | Date: | Sat, 09 Jul 2005 00:47:09 +0000 |
| Subject: | #33588 [Asn->Csd]: No RootDSE query possible | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-81605@lists.php.net to get a copy of this message | ||
ID: 33588
Updated by: sniper@php.net
Reported By: cajus at naasa dot net
-Status: Assigned
+Status: Closed
Bug Type: LDAP related
Operating System: Debian GNU/Linux
PHP Version: 5.0.4
Assigned To: sniper
New Comment:
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.
Previous Comments:
------------------------------------------------------------------------
[2005-07-06 13:55:09] cajus at naasa dot net
Description:
------------
I've found that searching the RootDSE does not work with
PHP, because the base is set to NULL when no base is
specified. This fact causes the LDAP library to look into
the systems ldap.conf to take the base from there - which
is probably not what we want.
The following patch fixes the problem:
--- php5-5.0.4/ext/ldap/ldap.c 2005-01-19
01:27:42.000000000 +0100
+++ php5-5.0.4/ext/ldap/ldap.c.fixed 2005-07-01
17:15:55.000000000 +0200
@@ -575,6 +575,7 @@
{
zval **link, **base_dn, **filter, **attrs, **attr,
**attrsonly, **sizelimit, **timelimit, **deref;
char *ldap_base_dn = NULL;
+ char *dummy_base_dn = "";
char *ldap_filter = NULL;
char **ldap_attrs = NULL;
ldap_linkdata *ld;
@@ -674,7 +675,7 @@
} else {
nbases = 0; /* this means string,
not array */
convert_to_string_ex(base_dn);
- ldap_base_dn =
Z_STRLEN_PP(base_dn) < 1 ? NULL : Z_STRVAL_PP(base_dn);
+ ldap_base_dn =
Z_STRLEN_PP(base_dn) < 1 ? dummy_base_dn :
Z_STRVAL_PP(base_dn);
}
if (Z_TYPE_PP(filter) == IS_ARRAY) {
@@ -713,7 +714,7 @@
zend_hash_get_current_data(Z_ARRVAL_PP(base_dn), (void
**)&entry);
zend_hash_move_forward(Z_ARRVAL_PP(base_dn));
convert_to_string_ex(entry);
- ldap_base_dn =
Z_STRLEN_PP(entry) < 1 ? NULL : Z_STRVAL_PP(entry);
+ ldap_base_dn =
Z_STRLEN_PP(entry) < 1 ? dummy_base_dn :
Z_STRVAL_PP(entry);
}
if (nfilters != 0) { /* filter an
array? */
zend_hash_get_current_data(Z_ARRVAL_PP(filter), (void
**)&entry);
@@ -756,7 +757,7 @@
/* fix to make null base_dn's work */
if (strlen(ldap_base_dn) < 1) {
- ldap_base_dn = NULL;
+ ldap_base_dn = dummy_base_dn;
}
ld = (ldap_linkdata *) zend_fetch_resource(link
TSRMLS_CC, -1, "ldap link", NULL, 1, le_link);
Reproduce code:
---------------
<?php
# ldapsearch -x -b '' -s base '(objectclass=*)' namingContexts
function get_naming_contexts($server, $admin, $password)
{
/* Build LDAP connection */
$ds= ldap_connect ($server);
if (!$ds) {
die ("Can't bind to LDAP. No check possible!");
}
ldap_set_option($ds, LDAP_OPT_PROTOCOL_VERSION, 3);
$r= ldap_bind ($ds, $admin, $password);
/* Get base to look for naming contexts */
$sr = @ldap_read ($ds, "", "objectClass=*",
array("namingContexts"));
$attr= @ldap_get_entries($ds,$sr);
print_r($attr);
}
get_naming_contexts("what.server.ever.org",
"cn=admindn,dc=whatever,dc=org", "secret");
?>
Expected result:
----------------
Array
(
[count] => 1
[0] => Array
(
[namingcontexts] => Array
(
[count] => 2
[0] => dc=whatever,dc=org
[1] => dc=whatever,dc=shell
)
[0] => namingcontexts
[count] => 1
[dn] =>
)
)
Actual result:
--------------
Array
(
[count] => 1
[0] => Array
(
[count] => 0
[dn] => dc=whatever,dc=org
)
)
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=33588&edit=1