No need for check since callback_name is always filled in.

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