Skip to content

fix(fpm): Fix FPM test harness object leaks reported under ASan and Valgrind. - #23494

Closed
zeriyoshi wants to merge 1 commit into
php:masterfrom
zeriyoshi:fpm_test_leak_fix
Closed

zeriyoshi wants to merge 1 commit into
php:masterfrom
zeriyoshi:fpm_test_leak_fix

Conversation

@zeriyoshi

Copy link
Copy Markdown
Contributor

This fix addresses an issue in clang-23 stable where running memory tests with ASan causes fpm to flag memory as unreleased.

@zeriyoshi
zeriyoshi requested a review from bukka as a code owner August 29, 2026 11:20
@zeriyoshi zeriyoshi changed the title fix: Fix FPM test harness object leaks reported under ASan and Valgrind. fix(fpm): Fix FPM test harness object leaks reported under ASan and Valgrind. Aug 29, 2026
@bukka

bukka commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Isn't this more engine issue that should be fixed there? There should be no leaks reported for a user space code so I don't think this is the right fix.

@zeriyoshi

Copy link
Copy Markdown
Contributor Author

That's right. I'll close this, sorry

@zeriyoshi zeriyoshi closed this Oct 28, 2026
@zeriyoshi
zeriyoshi deleted the fpm_test_leak_fix branch October 28, 2026 10:58
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