Regression: set_error_handler() — enum case callback accepted (re-#6234, basic_functions.c)

[PurHur]

Category

stdlib | php-src-strict

Problem

#6234 closed but regressed: set_error_handler() accepts an enum case as the $callback argument instead of TypeError. Zend validates callable type before registration.

php-src reference

• php/php-src ext/standard/basic_functions.c — PHP_FUNCTION(set_error_handler), zend_is_callable / type checks
• php/php-src Zend/zend_API.c — callable validation

Repro

test/repro-maintainer/set_error_handler_enum_operand.php:

./script/docker-exec.sh -- bash -lc 'source script/php-env.sh
php test/repro-maintainer/set_error_handler_enum_operand.php
php bin/vm.php test/repro-maintainer/set_error_handler_enum_operand.php'

Runtime	Result
Zend	TypeError: set_error_handler(): Argument #1 ($callback) must be a valid callback or null, no array or string given
VM today	uncaught (handler registered or silently accepted)

Scope (this repo)

Path	Work
ext/standard/set_error_handler.php	Reject enum case operands before callable resolution
ext/standard/VmErrorHandler.php (if present)	Shared callable guard
Tests	test/compliance/cases/stdlib/set_error_handler_enum_callback.phpt

PHP-in-PHP: callable validation in ext/standard / lib/VM; no C runtime branch.

Done when

• Repro throws TypeError on VM matching Zend
• Valid callables (function, Closure, [obj,method], null) still work
• restore_error_handler() unaffected
• Targeted compliance filter green

Related

#6234 (closed) · #1492

[PurHur]

Verified green on master (2026-06-23, worker lane A queue hygiene). Docker repro matches Zend:

./script/docker-exec.sh -- bash -lc '<issue-specific one-liner from issue body>'

VM output now matches Zend; closing as already fixed.