Don't allow separation in CallbackFilterIterator

As the name might suggest, this is a *filter* iterator. If you want to have a *map* iterator, write one, or use a generator.
php · Jul 7, 2020 · df8119d · df8119d
1 parent 632766a
commit df8119d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/ext/spl/spl_iterators.c b/ext/spl/spl_iterators.c
@@ -1785,7 +1785,7 @@ PHP_METHOD(CallbackFilterIterator, accept)
 	fci->retval = return_value;
 	fci->param_count = 3;
 	fci->params = params;
-	fci->no_separation = 0;
+	fci->no_separation = 1;
 
 	if (zend_call_function(fci, fcc) != SUCCESS || Z_ISUNDEF_P(return_value)) {
 		RETURN_FALSE;
@@ -1794,10 +1794,6 @@ PHP_METHOD(CallbackFilterIterator, accept)
 	if (EG(exception)) {
 		RETURN_THROWS();
 	}
-
-	/* zend_call_function may change args to IS_REF */
-	ZVAL_COPY_VALUE(&intern->current.data, &params[0]);
-	ZVAL_COPY_VALUE(&intern->current.key, &params[1]);
 }
 /* }}} */
 

diff --git a/ext/spl/tests/bug72051.phpt b/ext/spl/tests/bug72051.phpt
@@ -17,10 +17,10 @@ $data = $callbackTest->current();
 $callbackTest->next();
 print_r($data);
 ?>
---EXPECT--
+--EXPECTF--
+Warning: {closure}(): Argument #1 ($current) must be passed by reference, value given in %s on line %d
 Array
 (
     [0] => 1
     [1] => 2
-    [message] => Test message
 )