|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #41390 stream_wrapper_register() and underscore in $protocol
Submitted: 2007-05-14 21:39 UTC Modified: 2007-05-15 13:04 UTC
From: Assigned:
Status: Closed Package: Streams related
PHP Version: 5.2.2 OS: WinXP SP2
Private report: No CVE-ID: None
 [2007-05-14 21:39 UTC]
When I use underscores in $protocol parameter for stream_wrapper_register(), I get an error telling me that PHP was "Unable to register wrapper class ...". In main/streams/userspace. this error is thrown and one can find the comment "/* Should never happen */" above it.

Either the comment is wrong or there should be a check for the validity of the protocal name.


Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2007-05-15 03:12 UTC]
A protocol has to be A-Z 0-9 so an underscore isn't valid and is the cause of the error.

See php_register_url_stream_wrapper in main/streams/streams.c
 [2007-05-15 07:35 UTC]
In php_register_url_stream_wrapper the protocal is checked, in my case it isn't checked. And "Should never happen" is a rather bogus comment for a case that actually does happen, isn't it?
 [2007-05-15 12:20 UTC]
The comment is most likely wrong, the call fails because php_register_url_stream_wrapper_volatile returns FAILURE.

The reason it returns FAILURE because php_register_url_stream_wrapper_volatile then runs php_stream_wrapper_scheme_validate which fails because the protocol isn't /^[a-zA-Z0-9+.-]+$/

The error could well be clarified.
 [2007-05-15 13:04 UTC]
This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
Thank you for the report, and for helping us make PHP better.

PHP Copyright © 2001-2023 The PHP Group
All rights reserved.
Last updated: Fri Dec 01 17:01:28 2023 UTC