#41390 [Opn->Csd]: stream_wrapper_register() and underscore in $protocol

ID: 41390 Updated by: scottmac@php.net Reported By: wiesemann@php.net -Status: Open +Status: Closed Bug Type: Streams related Operating System: WinXP SP2 PHP Version: 5.2.2 New Comment: 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 http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2007-05-15 12:20:16] scottmac@php.net 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 07:35:22] wiesemann@php.net 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 03:12:41] scottmac@php.net 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-14 21:39:33] wiesemann@php.net Description: ------------ 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. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=41390&edit=1