Skip to content

[pull] master from php:master#563

Merged
pull[bot] merged 9 commits intoturkdevops:masterfrom
php:master
Nov 30, 2025
Merged

[pull] master from php:master#563
pull[bot] merged 9 commits intoturkdevops:masterfrom
php:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Nov 30, 2025

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 : )

Since the executor needs to be active at this point, the only way you
could get an UNDEF return value is by having an exception.
Therefore, `!EG(exception)` is always false.
The check doesn't make sense, remove it.
…lization

All other code caters to dereferencing array elements, except the
unserialize handler. This causes references to be present in the fixed
array even though this seems not intentional as reference assign is
otherwise impossible.
On 8.5+ this causes an assertion failure. On 8.3+ this causes references
to be present where they shouldn't be.

Closes GH-20616.
* PHP-8.3:
  Fix GH-20614: SplFixedArray incorrectly handles references in deserialization
* PHP-8.4:
  Fix GH-20614: SplFixedArray incorrectly handles references in deserialization
* PHP-8.5:
  Fix GH-20614: SplFixedArray incorrectly handles references in deserialization
@pull pull bot locked and limited conversation to collaborators Nov 30, 2025
@pull pull bot added the ⤵️ pull label Nov 30, 2025
@pull pull bot merged commit 1848bcd into turkdevops:master Nov 30, 2025
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.

1 participant