Bug #49391 ldap.c utilizing deprecated ldap_modify_s

diff --git a/NEWS b/NEWS
index 47079eb97f864ff9ffadfc785f176ea16c943d15..93db3f8793baab0bca3dfa32fa829eca43ec5011 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -27,6 +27,7 @@ PHP                                                                        NEWS
 - Fixed memory leak in stream_is_local(). (Felipe, Tony)
 - Fixed BC break in mime_content_type(), removes the content encoding. (Scott) 
 
+- Fixed bug #49391 (ldap.c utilizing deprecated ldap_modify_s). (Ilia)
 - Fixed bug #49372 (segfault in php_curl_option_curl). (Pierre)
 - Fixed bug #49306 (inside pdo_mysql default socket settings are ignored).
   (Ilia)

diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c
index 34dedaf72722e3e1c2b1073c3e0a641cd88b3750..2a1b9d12f7ba49805bcf060f4bba3d96e2388e02 100644 (file)
--- a/ext/ldap/ldap.c
+++ b/ext/ldap/ldap.c
@@ -1350,7 +1350,7 @@ static void php_ldap_do_modify(INTERNAL_FUNCTION_PARAMETERS, int oper)
                        RETVAL_FALSE;
                } else RETVAL_TRUE;
        } else {
-               if ((i = ldap_modify_s(ld->link, dn, ldap_mods)) != LDAP_SUCCESS) {
+               if ((i = ldap_modify_ext_s(ld->link, dn, ldap_mods, NULL, NULL)) != LDAP_SUCCESS) {
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "Modify: %s", ldap_err2string(i));
                        RETVAL_FALSE;
                } else RETVAL_TRUE;