addressing GH-18799 remarks

Author: devnexen

ext/pdo_sqlite/pdo_sqlite.c

@@ -385,10 +385,10 @@ static int php_sqlite_collation_callback(void *context, int string1_len, const v
 			zend_type_error("%s(): Return value of the collation callback must be of type int, %s returned",
 				ZSTR_VAL(func_name), zend_zval_value_name(&retval));
 			zend_string_release(func_name);
-			ret = FAILURE;
+			zval_ptr_dtor(&retval);
+			return FAILURE;
 		}
 		ret = ZEND_NORMALIZE_BOOL(Z_LVAL(retval));
-		zval_ptr_dtor(&retval);
 	}
 
 	return ret;

ext/pdo_sqlite/sqlite_driver.c

@@ -492,10 +492,10 @@ static int php_sqlite3_collation_callback(void *context, int string1_len, const
 			zend_type_error("%s(): Return value of the collation callback must be of type int, %s returned",
 				ZSTR_VAL(func_name), zend_zval_value_name(&retval));
 			zend_string_release(func_name);
-			ret = FAILURE;
+			zval_ptr_dtor(&retval);
+			return FAILURE;
 		}
 		ret = ZEND_NORMALIZE_BOOL(Z_LVAL(retval));
-		zval_ptr_dtor(&retval);
 	}
 
 	return ret;