com php-src: Removed deprecated function calls from ldap module: ext/ldap/ldap.c

Commit: 5250fc772d291f47bf6422dd5a72f6812be1b2e2 Author: Côme Bernigaud <mcmic@php.net> Tue, 23 Jun 2015 16:44:48 +0200 Parents: ea49be9b371c2e6ce2efaf88a89fc3740d8c6328 Branches: PHP-5.6 Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=5250fc772d291f47bf6422dd5a72f6812be1b2e2 Log: Removed deprecated function calls from ldap module ldap_search -> ldap search_ext ldap_search_s ->_ldap_search_ext_s ldap_add_s -> ldap_add_ext_s ldap_delete_s -> ldap_delete_ext_s ldap_compare_s -> ldap_compare_ext_s ldap_value_free -> ldap_memvfree Only thing left is ldap_sort_entries, no replacement Changed paths: M ext/ldap/ldap.c Diff: diff --git a/ext/ldap/ldap..c b/ext/ldap/ldap.c index 62ca99e..e2bcd17 100644 --- a/ext/ldap/ldap.c +++ b/ext/ldap/ldap.c @@ -783,7 +783,7 @@ static void php_ldap_do_search(INTERNAL_FUNCTION_PARAMETERS, int scope) php_set_opts(ld->link, ldap_sizelimit, ldap_timelimit, ldap_deref, &old_ldap_sizelimit, &old_ldap_timelimit, &old_ldap_deref); /* Run the actual search */ - rcs[i] = ldap_search(ld->link, ldap_base_dn, scope, ldap_filter, ldap_attrs, ldap_attrsonly); + ldap_search_ext(ld->link, ldap_base_dn, scope, ldap_filter, ldap_attrs, ldap_attrsonly, NULL, NULL, NULL, ldap_sizelimit, &rcs[i]); lds[i] = ld; zend_hash_move_forward(Z_ARRVAL_P(link)); } @@ -825,7 +825,7 @@ cleanup_parallel: php_set_opts(ld->link, ldap_sizelimit, ldap_timelimit, ldap_deref, &old_ldap_sizelimit, &old_ldap_timelimit, &old_ldap_deref); /* Run the actual search */ - errno = ldap_search_s(ld->link, ldap_base_dn, scope, ldap_filter, ldap_attrs, ldap_attrsonly, &ldap_res); + errno = ldap_search_ext_s(ld->link, ldap_base_dn, scope, ldap_filter, ldap_attrs, ldap_attrsonly, NULL, NULL, NULL, ldap_sizelimit, &ldap_res); if (errno != LDAP_SUCCESS && errno != LDAP_SIZELIMIT_EXCEEDED @@ -1297,7 +1297,7 @@ PHP_FUNCTION(ldap_explode_dn) } /* ldap_value_free() is deprecated */ - ber_memvfree((void **)ldap_value); + ldap_memvfree((void **)ldap_value); } /* }}} */ @@ -1421,7 +1421,7 @@ static void php_ldap_do_modify(INTERNAL_FUNCTION_PARAMETERS, int oper) /* check flag to see if do_mod was called to perform full add , gerrit thomson */ if (is_full_add == 1) { - if ((i = ldap_add_s(ld->link, dn, ldap_mods)) != LDAP_SUCCESS) { + if ((i = ldap_add_ext_s(ld->link, dn, ldap_mods, NULL, NULL)) != LDAP_SUCCESS) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Add: %s", ldap_err2string(i)); RETVAL_FALSE; } else RETVAL_TRUE; @@ -1498,7 +1498,7 @@ PHP_FUNCTION(ldap_delete) ZEND_FETCH_RESOURCE(ld, ldap_linkdata *, &link, -1, "ldap link", le_link); - if ((rc = ldap_delete_s(ld->link, dn)) != LDAP_SUCCESS) { + if ((rc = ldap_delete_ext_s(ld->link, dn, NULL, NULL)) != LDAP_SUCCESS) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Delete: %s", ldap_err2string(rc)); RETURN_FALSE; } @@ -1921,9 +1921,13 @@ PHP_FUNCTION(ldap_compare) return; } + struct berval lvalue; + lvalue.bv_val = value; + lvalue.bv_len = value_len; + ZEND_FETCH_RESOURCE(ld, ldap_linkdata *, &link, -1, "ldap link", le_link); - errno = ldap_compare_s(ld->link, dn, attr, value); + errno = ldap_compare_ext_s(ld->link, dn, attr, &lvalue, NULL, NULL); switch (errno) { case LDAP_COMPARE_TRUE: @@ -2288,7 +2292,7 @@ PHP_FUNCTION(ldap_parse_result) add_next_index_string(referrals, *refp, 1); refp++; } - ldap_value_free(lreferrals); + ldap_memvfree((void**)lreferrals); } case 5: zval_dtor(errmsg); @@ -2399,7 +2403,7 @@ PHP_FUNCTION(ldap_parse_reference) add_next_index_string(referrals, *refp, 1); refp++; } - ldap_value_free(lreferrals); + ldap_memvfree((void**)lreferrals); } RETURN_TRUE; }