Skip to content

[pull] master from php:master#768

Merged
pull[bot] merged 2 commits intoturkdevops:masterfrom
php:master
Mar 8, 2026
Merged

[pull] master from php:master#768
pull[bot] merged 2 commits intoturkdevops:masterfrom
php:master

Conversation

@pull
Copy link

@pull pull bot commented Mar 8, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

kamil-tekiela and others added 2 commits March 7, 2026 22:09
* Stop reporting unhelpful values in mysqli_get_charset

* Remove element "comment" as it has been undocumented
…21366)

ReflectionMethod::invoke() (and invokeArgs()) for Closure::__invoke()
incorrectly accepted any Closure object, not just the one the
ReflectionMethod was created from. This happened because all Closures
share a single zend_ce_closure class entry, so the instanceof_function()
check always passed.

Fix: store the original Closure object in intern->obj during
ReflectionMethod construction, then compare object identity in
reflection_method_invoke() to reject different Closure instances.

Closes GH-21362
@pull pull bot locked and limited conversation to collaborators Mar 8, 2026
@pull pull bot added the ⤵️ pull label Mar 8, 2026
@pull pull bot merged commit 53e31d5 into turkdevops:master Mar 8, 2026
1 check failed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants