RE: [PHP-DEV] Adding path_len to all stream functions in trunk
| From: | Andi Gutmans | Date: | Mon, 15 Nov 2010 05:35:54 +0000 |
| Subject: | RE: [PHP-DEV] Adding path_len to all stream functions in trunk | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-50199@lists.php.net to get a copy of this message | ||
Hi Rasmus,
Hope I understood the problem correctly. If not, this answer won't make sense :)
I do not see a major problem in passing path_len but wonder how much it'd actually solve as we
end up calling OS APIs that do not accept path_len, no? I assume we don't want to start
searching all these strings for invalid chars before we pass them to the OS.
Andi
> -----Original Message-----
> From: Rasmus Lerdorf [mailto:rasmus@lerdorf.com]
> Sent: Sunday, November 14, 2010 11:54 AM
> To: internals
> Subject: [PHP-DEV] Adding path_len to all stream functions in trunk
>
> I think we need to pass along the string length to all the stream functions to
> maintain binary string safety through this code. This would fix annoying
> problems like http://bugs.php.net/39863 and a bunch of similar
> issues.
> Obviously not something we can do in 5.3 without breaking binary compatibility
> though. Although we might be able to do something if we assume only chars
> valid in the current charset is valid in file paths.
>
> Anybody have any other thoughts on this one?
>
> -Rasmus
>
> --
> PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit:
> http://www.php.net/unsub.php