Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Dec 12, 2024

See Commits and Changes for more details.


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

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

iluuu1994 and others added 7 commits December 12, 2024 13:10
op1 of ZEND_MATCH_ERROR, which refers to the match expression, is not freed by
MATCH_ERROR itself. Instead, it is freed by ZEND_HANDLE_EXCEPTION. For normal
control flow, a FREE is placed at the end of the match expression.

Since FREE may appear after MATCH_ERROR in the opcode sequence, we need to
correctly handle op1 of MATCH_ERROR as alive.

Fixes GH-17106
Closes GH-17108
* PHP-8.3:
  Fix ZEND_MATCH_ERROR misoptimization
* PHP-8.4:
  Fix ZEND_MATCH_ERROR misoptimization
We don't show succeeding tests in the summary, and for all intents and purposes,
these tests have succeeded, in that they behave as expected. I've seen the
output confuse people on multiple occasions, for example GH-17105.

Closes GH-17109
* PHP-8.3:
  Hide xfail/xleak test summary
* PHP-8.4:
  Hide xfail/xleak test summary
…2. (#17123)

For SO_SNDTIMEO/SO_RCVTIMEO, normally the timeout ought to be of
DWORD/unsigned long.
@pull pull bot added the ⤵️ pull label Dec 12, 2024
@pull pull bot merged commit 07cd468 into wudi:master Dec 12, 2024
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.

2 participants