archive/php-src
1
0
Fork 0
mirror of https://github.com/php/php-src.git synced 2025-08-18 23:18:56 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

Merge branch 'php-7.0'

Browse source 
* php-7.0:
  Revert "Added back support for undocummented host:port syntax"
  Revert "Using default port in ldap_connect when NULL is passed (this was the behavior prior to 5.6.11)"
This commit is contained in:
Côme Bernigaud 2015-11-23 03:15:46 +01:00
commit 56b584491b
2 changed files with 1 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

10
ext/ldap/ldap.c
View file

@ -345,9 +345,6 @@ PHP_FUNCTION(ldap_connect)
RETURN_FALSE;
}
#endif
if (!port) {
port = LDAP_PORT;
}
if (LDAPG(max_links) != -1 && LDAPG(num_links) >= LDAPG(max_links)) {
php_error_docref(NULL, E_WARNING, "Too many open links (%pd)", LDAPG(num_links));
@ -368,12 +365,7 @@ PHP_FUNCTION(ldap_connect)
}
url = emalloc(urllen);
if (host && (strchr(host, ':') != NULL)) {
/* Legacy support for host:port */
snprintf( url, urllen, "ldap://%s", host );
} else {
snprintf( url, urllen, "ldap://%s:%ld", host ? host : "", port );
}
snprintf( url, urllen, "ldap://%s:%ld", host ? host : "", port );
}
#ifdef LDAP_API_FEATURE_X_OPENLDAP

5
ext/ldap/tests/ldap_connect_variation.phpt
View file

@ -28,10 +28,6 @@ var_dump($link);
// bad hostname (connect should work, not bind)
$link = ldap_connect("nonexistent" . $host);
var_dump($link);
// Legacy host:port syntax
$link = ldap_connect("$host:$port");
var_dump($link);
?>
===DONE===
--EXPECTF--
@ -40,5 +36,4 @@ resource(%d) of type (ldap link)
resource(%d) of type (ldap link)
resource(%d) of type (ldap link)
resource(%d) of type (ldap link)
resource(%d) of type (ldap link)
===DONE===