Skip to content

Fix #[\Deprecated] for __call() and __callStatic() #17592

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jan 27, 2025

Conversation

TimWolla
Copy link
Member

@TimWolla TimWolla commented Jan 27, 2025

Fixes GH-17597.

/cc @beberlei

@TimWolla TimWolla force-pushed the deprecated-magic-call branch from 321bcde to 22dc09c Compare January 27, 2025 11:30
@TimWolla TimWolla merged commit f37b165 into php:PHP-8.4 Jan 27, 2025
10 checks passed
TimWolla added a commit that referenced this pull request Jan 27, 2025
* PHP-8.4:
  Fix `#[\Deprecated]` for `__call()` and `__callStatic()` (#17592)
@TimWolla TimWolla deleted the deprecated-magic-call branch January 27, 2025 12:42
nielsdos added a commit to nielsdos/php-src that referenced this pull request Feb 21, 2025
…4.3 to 8.4.4

This regressed in phpGH-17592.
The function is with its attributes HashTable* is copied in
zend_get_closure_invoke_method() but its refcount is not increased.
This caused a crash in the Symfony demo page.
nielsdos added a commit that referenced this pull request Feb 24, 2025
… to 8.4.4

This regressed in GH-17592.
The function is with its attributes HashTable* is copied in
zend_get_closure_invoke_method() but its refcount is not increased.
This caused a crash in the Symfony demo page.

Closes GH-17880.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants