[2004-11-14 18:26 UTC] bekkers at gmail dot com

Description:
------------
The problem is that ZEND_METHOD(reflection_function, getStaticVariables) in zend_reflection_api.c does not check wether the function is userdefined or not. Wich apperently makes Apache crash.

I figured the function should look something like this:

/* {{{ proto public array ReflectionFunction::getStaticVariables()
   Returns an associative array containing this function's static variables and their values */
ZEND_METHOD(reflection_function, getStaticVariables)
{
	zval *tmp_copy;
	reflection_object *intern;
	zend_function *fptr;
	
	METHOD_NOTSTATIC_NUMPARAMS(0);
	GET_REFLECTION_OBJECT_PTR(fptr);

	/* Return an empty array in case no static variables exist */
	array_init(return_value);
	if (fptr->type == ZEND_USER_FUNCTION && fptr->op_array.static_variables != NULL) {
		zend_hash_copy(Z_ARRVAL_P(return_value), fptr->op_array.static_variables, (copy_ctor_func_t) zval_add_ref, (void *) &tmp_copy, sizeof(zval *));
	}
}

But I'm not shure and as I haven't (and can't) test it.

Reproduce code:
---------------
<?php
$R = new ReflectionFunction('phpversion');
$R->getStaticVariables();
?>

Expected result:
----------------
The output should be nothing. (Getting the static variables from a C function does not sound logical)

Actual result:
--------------
Apache crashes

 [2004-11-14 19:10 UTC] helly@php.net

This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.