Revert previous commit - callback_name does need to be checked because

it is set conditional on is_callable_replace.

diff --git a/ext/pcre/php_pcre.c b/ext/pcre/php_pcre.c
index b6db493ee37a0e1921d8e8cf58184aa599839204..4344591cd65ff09aeba5feaf9227a7650f9e0e6c 100644 (file)
--- a/ext/pcre/php_pcre.c
+++ b/ext/pcre/php_pcre.c
@@ -989,7 +989,8 @@ static void preg_replace_impl(INTERNAL_FUNCTION_PARAMETERS, zend_bool is_callabl
                zval_copy_ctor(return_value);
                return;
        }
-       efree(callback_name);
+       if (callback_name)
+               efree(callback_name);
 
        SEPARATE_ZVAL(regex);
        SEPARATE_ZVAL(subject);