Skip to content

fuzzer: make ZTS build possible#22713

Open
henderkes wants to merge 1 commit into
php:masterfrom
henderkes:fix/fuzzer-zts
Open

fuzzer: make ZTS build possible#22713
henderkes wants to merge 1 commit into
php:masterfrom
henderkes:fix/fuzzer-zts

Conversation

@henderkes

Copy link
Copy Markdown
Contributor

No description provided.

@ndossche

Copy link
Copy Markdown
Member

Have you tested this for a while? Last time I tried this, it fell over in ext/date initialization

@henderkes

Copy link
Copy Markdown
Contributor Author

Yes, it's working without any apparent issues. I can look at the history (I assume this was once enabled but was disabled due to issues?) if you want.

@henderkes

Copy link
Copy Markdown
Contributor Author

https://github.com/php/php-src/blob/ca3444b1d8c7c5c6ddb096d93d48125e915f4ae5/sapi/fuzzer/fuzzer-sapi.c

It never called php_tsrm_startup() so the first extensions minit would naturally fail when trying to deref the not-filled thread globals.

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