From ca476ecaa47cf6017354c8ddb4ca464f1fb3533b Mon Sep 17 00:00:00 2001 From: Stig Venaas Date: Tue, 13 Nov 2001 17:10:08 +0000 Subject: [PATCH] Removed some old cruft (some commented code and non-used globals), fixed a proto typo, and make sure ldap_base_dn and ldap_filter are initialized in php_ldap_do_search() --- ext/ldap/ldap.c | 27 ++++----------------------- ext/ldap/php_ldap.h | 2 -- 2 files changed, 4 insertions(+), 25 deletions(-) diff --git a/ext/ldap/ldap.c b/ext/ldap/ldap.c index 0835691e6a..e03612a118 100644 --- a/ext/ldap/ldap.c +++ b/ext/ldap/ldap.c @@ -161,7 +161,6 @@ static void _free_ldap_result(zend_rsrc_list_entry *rsrc TSRMLS_DC) */ PHP_INI_BEGIN() STD_PHP_INI_ENTRY_EX("ldap.max_links", "-1", PHP_INI_SYSTEM, OnUpdateInt, max_links, zend_ldap_globals, ldap_globals, display_link_numbers) - STD_PHP_INI_ENTRY("ldap.base_dn", NULL, PHP_INI_ALL, OnUpdateString, base_dn, zend_ldap_globals, ldap_globals) PHP_INI_END() /* }}} */ @@ -303,16 +302,12 @@ PHP_FUNCTION(ldap_connect) int authmode; int ssl=0; #endif - /* char *hashed_details; - int hashed_details_length;*/ LDAP *ldap; switch(ZEND_NUM_ARGS()) { case 0: host = NULL; port = 0; - /* hashed_details = estrndup("ldap_", 5); - hashed_details_length = 4+1; */ break; case 1: { @@ -325,10 +320,6 @@ PHP_FUNCTION(ldap_connect) convert_to_string_ex(yyhost); host = Z_STRVAL_PP(yyhost); port = 389; /* Default port */ - - /* hashed_details_length = Z_STRLEN_P(yyhost)+4+1; - hashed_details = emalloc(hashed_details_length+1); - sprintf(hashed_details, "ldap_%s", Z_STRVAL_P(yyhost));*/ } break; @@ -343,11 +334,6 @@ PHP_FUNCTION(ldap_connect) host = Z_STRVAL_PP(yyhost); convert_to_long_ex(yyport); port = Z_LVAL_PP(yyport); - - /* Do we need to take care of hosts running multiple LDAP servers ? */ - /* hashed_details_length = Z_STRLEN_P(yyhost)+4+1; - hashed_details = emalloc(hashed_details_length+1); - sprintf(hashed_details, "ldap_%s", Z_STRVAL_P(yyhost));*/ } break; #ifdef HAVE_ORALDAP @@ -503,7 +489,7 @@ PHP_FUNCTION(ldap_unbind) } /* }}} */ -/* {{{ hp_set_opts +/* {{{ php_set_opts */ static void php_set_opts(LDAP *ldap, int sizelimit, int timelimit, int deref) { @@ -541,7 +527,8 @@ static void php_set_opts(LDAP *ldap, int sizelimit, int timelimit, int deref) static void php_ldap_do_search(INTERNAL_FUNCTION_PARAMETERS, int scope) { pval **link, **base_dn, **filter, **attrs, **attr, **attrsonly, **sizelimit, **timelimit, **deref; - char *ldap_base_dn, *ldap_filter; + char *ldap_base_dn = NULL; + char *ldap_filter = NULL; char **ldap_attrs = NULL; LDAP *ldap; LDAPMessage *ldap_res; @@ -1335,7 +1322,6 @@ static void php_ldap_do_modify(INTERNAL_FUNCTION_PARAMETERS, int oper) ldap_mods[i]->mod_bvalues = emalloc((num_values + 1) * sizeof(struct berval *)); /* allow for arrays with one element, no allowance for arrays with none but probably not required, gerrit thomson. */ -/* if (num_values == 1) {*/ if ((num_values == 1) && (Z_TYPE_PP(value) != IS_ARRAY)) { convert_to_string_ex(value); ldap_mods[i]->mod_bvalues[0] = (struct berval *) emalloc (sizeof(struct berval)); @@ -1362,8 +1348,7 @@ static void php_ldap_do_modify(INTERNAL_FUNCTION_PARAMETERS, int oper) ldap_mods[num_attribs] = NULL; /* check flag to see if do_mod was called to perform full add , gerrit thomson */ -/* if (oper == LDAP_MOD_ADD) { */ - if (is_full_add == 1) { + if (is_full_add == 1) { if (ldap_add_s(ldap, ldap_dn, ldap_mods) != LDAP_SUCCESS) { ldap_perror(ldap, "LDAP"); php_error(E_WARNING, "LDAP: add operation could not be completed."); @@ -1397,7 +1382,6 @@ errexit: PHP_FUNCTION(ldap_add) { /* use a newly define parameter into the do_modify so ldap_mod_add can be used the way it is supposed to be used , Gerrit THomson */ - /* php_ldap_do_modify(INTERNAL_FUNCTION_PARAM_PASSTHRU, LDAP_MOD_ADD);*/ php_ldap_do_modify(INTERNAL_FUNCTION_PARAM_PASSTHRU, PHP_LD_FULL_ADD); } /* }}} */ @@ -1437,9 +1421,6 @@ PHP_FUNCTION(ldap_mod_del) } /* }}} */ -/* end of attribute based functions , gerrit thomson */ - - /* {{{ proto int ldap_delete(int link, string dn) Delete an entry from a directory */ PHP_FUNCTION(ldap_delete) diff --git a/ext/ldap/php_ldap.h b/ext/ldap/php_ldap.h index 140111d71f..ab6a2bc98d 100644 --- a/ext/ldap/php_ldap.h +++ b/ext/ldap/php_ldap.h @@ -98,10 +98,8 @@ PHP_FUNCTION(ldap_8859_to_t61); #endif ZEND_BEGIN_MODULE_GLOBALS(ldap) - long default_link; long num_links; long max_links; - char *base_dn; ZEND_END_MODULE_GLOBALS(ldap) #ifdef ZTS -- 2.40.0