-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Performance regression in PHP 8.3 #11507
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
Labels
Comments
https://3v4l.org/WciJK shows that previously it was a constant time It makes 1 test of Drupal to hang forever with following strace |
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
Jun 22, 2023
When the code was moved to solve the uaf for memory overflow, this caused the refcount to be higher than one in some self-concatenation scenarios. This in turn causes quadratic time performance problems when these concatenations happen in a loop.
I was about to post a PR. It's actually bcuz of 7790ee8 |
@nielsdos Ah I see, that's my bad, thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The following code:
Resulted in this output in previous versions:
But I'm getting this output instead:
PHP Version
PHP 8.3.0-alpha2
Operating System
Alpinelinux, Ubuntu
The text was updated successfully, but these errors were encountered: