summaryrefslogtreecommitdiff
path: root/Zend/zend_builtin_functions.c
diff options
context:
space:
mode:
authorDmitry Stogov <dmitry@zend.com>2013-03-19 15:45:48 +0400
committerDmitry Stogov <dmitry@zend.com>2013-03-19 15:45:48 +0400
commit984561cfa81adfd893833bce506544349b04c803 (patch)
tree4222ef8e5a90ee24811731d1001d4afac356d480 /Zend/zend_builtin_functions.c
parent84630a1109084507bc0c78301db831328229f1cb (diff)
downloadphp-git-984561cfa81adfd893833bce506544349b04c803.tar.gz
Partial fix for bug #64239
Diffstat (limited to 'Zend/zend_builtin_functions.c')
-rw-r--r--Zend/zend_builtin_functions.c37
1 files changed, 24 insertions, 13 deletions
diff --git a/Zend/zend_builtin_functions.c b/Zend/zend_builtin_functions.c
index 1e85785c40..dcb18182e1 100644
--- a/Zend/zend_builtin_functions.c
+++ b/Zend/zend_builtin_functions.c
@@ -1025,6 +1025,13 @@ ZEND_FUNCTION(get_object_vars)
}
/* }}} */
+static int same_name(const char *key, const char *name, zend_uint name_len)
+{
+ char *lcname = zend_str_tolower_dup(name, name_len);
+ int ret = memcmp(lcname, key, name_len) == 0;
+ efree(lcname);
+ return ret;
+}
/* {{{ proto array get_class_methods(mixed class)
Returns an array of method names for class or class instance. */
@@ -1072,14 +1079,26 @@ ZEND_FUNCTION(get_class_methods)
uint len = strlen(mptr->common.function_name);
/* Do not display old-style inherited constructors */
- if ((mptr->common.fn_flags & ZEND_ACC_CTOR) == 0 ||
- mptr->common.scope == ce ||
- zend_hash_get_current_key_ex(&ce->function_table, &key, &key_len, &num_index, 0, &pos) != HASH_KEY_IS_STRING ||
- zend_binary_strcasecmp(key, key_len-1, mptr->common.function_name, len) == 0) {
-
+ if (zend_hash_get_current_key_ex(&ce->function_table, &key, &key_len, &num_index, 0, &pos) != HASH_KEY_IS_STRING) {
MAKE_STD_ZVAL(method_name);
ZVAL_STRINGL(method_name, mptr->common.function_name, len, 1);
zend_hash_next_index_insert(return_value->value.ht, &method_name, sizeof(zval *), NULL);
+ } else if ((mptr->common.fn_flags & ZEND_ACC_CTOR) == 0 ||
+ mptr->common.scope == ce ||
+ zend_binary_strcasecmp(key, key_len-1, mptr->common.function_name, len) == 0) {
+
+ if (mptr->type == ZEND_USER_FUNCTION &&
+ *mptr->op_array.refcount > 1 &&
+ (len != key_len - 1 ||
+ !same_name(key, mptr->common.function_name, len))) {
+ MAKE_STD_ZVAL(method_name);
+ ZVAL_STRINGL(method_name, key, key_len - 1, 1);
+ zend_hash_next_index_insert(return_value->value.ht, &method_name, sizeof(zval *), NULL);
+ } else {
+ MAKE_STD_ZVAL(method_name);
+ ZVAL_STRINGL(method_name, mptr->common.function_name, len, 1);
+ zend_hash_next_index_insert(return_value->value.ht, &method_name, sizeof(zval *), NULL);
+ }
}
}
zend_hash_move_forward_ex(&ce->function_table, &pos);
@@ -1640,14 +1659,6 @@ ZEND_FUNCTION(restore_exception_handler)
}
/* }}} */
-static int same_name(const char *key, const char *name, zend_uint name_len)
-{
- char *lcname = zend_str_tolower_dup(name, name_len);
- int ret = memcmp(lcname, key, name_len) == 0;
- efree(lcname);
- return ret;
-}
-
static int copy_class_or_interface_name(zend_class_entry **pce TSRMLS_DC, int num_args, va_list args, zend_hash_key *hash_key)
{
zval *array = va_arg(args, zval *);