Skip to content

Commit ed4de18

Browse files
committed
Fix null byte in LDAP bindings
1 parent 7f43aeb commit ed4de18

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

ext/ldap/ldap.c

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -399,6 +399,16 @@ PHP_FUNCTION(ldap_bind)
399399
RETURN_FALSE;
400400
}
401401

402+
if (ldap_bind_dn != NULL && memchr(ldap_bind_dn, '\0', ldap_bind_dnlen) != NULL) {
403+
php_error_docref(NULL TSRMLS_CC, E_WARNING, "DN contains a null byte");
404+
RETURN_FALSE;
405+
}
406+
407+
if (ldap_bind_pw != NULL && memchr(ldap_bind_pw, '\0', ldap_bind_pwlen) != NULL) {
408+
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Password contains a null byte");
409+
RETURN_FALSE;
410+
}
411+
402412
ZEND_FETCH_RESOURCE(ld, ldap_linkdata *, &link, -1, "ldap link", le_link);
403413

404414
if ((rc = ldap_bind_s(ld->link, ldap_bind_dn, ldap_bind_pw, LDAP_AUTH_SIMPLE)) != LDAP_SUCCESS) {

0 commit comments

Comments
 (0)