From: Felipe Pena Date: Thu, 9 Jun 2011 22:45:35 +0000 (+0000) Subject: - Fixed bug #54347 (reflection_extension does not lowercase module function name) X-Git-Tag: php-5.4.0alpha1~64 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cee9b6603c243d8ac0b27a1b1dff0839e8c07fa9;p=php - Fixed bug #54347 (reflection_extension does not lowercase module function name) patch by: laruence at yahoo dot com dot cn --- diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c index 1df797b664..57f8f48c30 100644 --- a/ext/reflection/php_reflection.c +++ b/ext/reflection/php_reflection.c @@ -1111,13 +1111,18 @@ static void _extension_string(string *str, zend_module_entry *module, char *inde /* Is there a better way of doing this? */ while (func->fname) { - if (zend_hash_find(EG(function_table), func->fname, strlen(func->fname) + 1, (void**) &fptr) == FAILURE) { + int fname_len = strlen(func->fname); + char *lc_name = zend_str_tolower_dup(func->fname, fname_len); + + if (zend_hash_find(EG(function_table), lc_name, fname_len + 1, (void**) &fptr) == FAILURE) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "Internal error: Cannot find extension function %s in global function table", func->fname); func++; + efree(lc_name); continue; } _function_string(str, fptr, NULL, " " TSRMLS_CC); + efree(lc_name); func++; } string_printf(str, "%s }\n", indent);