From f656c6fc70dedb3a2b62e9604396a1b82e32314c Mon Sep 17 00:00:00 2001 From: Dmitry Stogov Date: Wed, 1 Oct 2008 07:30:31 +0000 Subject: Fixed bug #46205 (Closure - Memory leaks when ReflectionException is thrown) --- ext/reflection/php_reflection.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'ext/reflection/php_reflection.c') diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c index 73cdbea5d4..aac558d993 100644 --- a/ext/reflection/php_reflection.c +++ b/ext/reflection/php_reflection.c @@ -1975,6 +1975,12 @@ ZEND_METHOD(reflection_parameter, __construct) if (Z_TYPE_PP(parameter) == IS_LONG) { position= Z_LVAL_PP(parameter); if (position < 0 || (zend_uint)position >= fptr->common.num_args) { + if (fptr->common.fn_flags & ZEND_ACC_CALL_VIA_HANDLER) { + if (fptr->type != ZEND_OVERLOADED_FUNCTION) { + efree(fptr->common.function_name); + } + efree(fptr); + } _DO_THROW("The parameter specified by its offset could not be found"); /* returns out of this function */ } @@ -1990,6 +1996,12 @@ ZEND_METHOD(reflection_parameter, __construct) } } if (position == -1) { + if (fptr->common.fn_flags & ZEND_ACC_CALL_VIA_HANDLER) { + if (fptr->type != ZEND_OVERLOADED_FUNCTION) { + efree(fptr->common.function_name); + } + efree(fptr); + } _DO_THROW("The parameter specified by its name could not be found"); /* returns out of this function */ } -- cgit v1.2.1