Are you sure about that?
Seems to me that this is one case that really should be protected by
allow_url_fopen, as it allows specifying arbitrary data in the
filename.
While this is not quite such a big deal in HEAD, it is in PHP 5.2.
I think this needs further analysis and discussion before we roll PHP 5.2.

--Wez.

On 5/14/06, Marcus Boerger <[EMAIL PROTECTED]> wrote:
helly           Sun May 14 19:13:17 2006 UTC

  Modified files:
    /php-src/main/streams       memory.c
  Log:
  - No need to have this controlled by allow_url_fopen

http://cvs.php.net/viewcvs.cgi/php-src/main/streams/memory.c?r1=1.19&r2=1.20&diff_format=u
Index: php-src/main/streams/memory.c
diff -u php-src/main/streams/memory.c:1.19 php-src/main/streams/memory.c:1.20
--- php-src/main/streams/memory.c:1.19  Sun May 14 01:39:24 2006
+++ php-src/main/streams/memory.c       Sun May 14 19:13:17 2006
@@ -16,7 +16,7 @@
    +----------------------------------------------------------------------+
  */

-/* $Id: memory.c,v 1.19 2006/05/14 01:39:24 helly Exp $ */
+/* $Id: memory.c,v 1.20 2006/05/14 19:13:17 helly Exp $ */

 #define _GNU_SOURCE
 #include "php.h"
@@ -701,7 +701,7 @@
 php_stream_wrapper php_stream_rfc2397_wrapper =        {
        &php_stream_rfc2397_wops,
        NULL,
-       1, /* is_url */
+       0, /* is_url */
 };

 /*

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to