[php-src] PR #23583: sapi/phpdbg: restore the DEFAULT_SLASH check on the module path
| From: | darkdi | Date: | Sat, 05 Sep 2026 10:02:33 +0000 |
| Subject: | [php-src] PR #23583: sapi/phpdbg: restore the DEFAULT_SLASH check on the module path | ||
| Groups: | php.git-pulls | ||
| Request: | Send a blank email to git-pulls+get-38325@lists.php.net to get a copy of this message | ||
Pull Request: https://github.com/php/php-src/pull/23583
Author: darkdi
> Before GH-21578 this guard read
strchr(*path, '/') || strchr(*path,
> DEFAULT_SLASH). The rewrite onto memchr() kept the first half and copied it into the
> second, so both now look for '/', and the DEFAULT_SLASH half is gone. On Windows that
> means ext C:\path\php_foo.dll is not treated as a path at all,
> extension_dir gets glued in front of it and the load fails. ext/standard/dl.c still has the pair
> intact, which is what the phpdbg code was written from. Only master carries this, 8.4 and 8.5 still
> have the strchr version, and I have not touched the NEWS file since nothing released is affected.
Disclosure per CONTRIBUTING: the paragraph above was drafted with an LLM (Claude Code). The finding
came from a static check of my own and I read the code and the history by hand.
🤖 Generated with [Claude Code](https://claude.com/claude-code)