Skip to content

Fixes leaked System V shared-memory segments in tests - #22911

Open
NickSdot wants to merge 2 commits into
php:masterfrom
NickSdot:fix/test-ipc-cleanup
Open

Fixes leaked System V shared-memory segments in tests#22911
NickSdot wants to merge 2 commits into
php:masterfrom
NickSdot:fix/test-ipc-cleanup

Conversation

@NickSdot

Copy link
Copy Markdown
Contributor

The two tests leave System V IPC segments behind after successful execution. Repeated test-suite runs can exhaust IPC limits and cause unrelated SHMOP/SysV tests to fail.

@LamentXU123 LamentXU123 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks correct. Thank you.
But since nobody is responsible for these extensions now. I would like to hear from Ilia as a veteran php core dev. (or others, if possible :) )

@LamentXU123
LamentXU123 requested a review from iliaal July 29, 2026 16:58

@iliaal iliaal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

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.

3 participants