MFPECL: fixed PECL Bug #14646: phar error message unclear with php stream wrappers

diff --git a/ext/phar/phar.c b/ext/phar/phar.c
index b2f6a82adea786101c7eca058322ed338811180e..4173cf0aab63560e65c87665978ecf4b6da12216 100644 (file)
--- a/ext/phar/phar.c
+++ b/ext/phar/phar.c
@@ -1262,7 +1262,11 @@ int phar_open_or_create_filename(char *fname, int fname_len, char *alias, int al
        /* next try to create a new file */
        if (FAILURE == phar_detect_phar_fname_ext(fname, fname_len, &ext_str, &ext_len, !is_data, 1, 1 TSRMLS_CC)) {
                if (error) {
-                       spprintf(error, 0, "Cannot create phar '%s', file extension (or combination) not recognised", fname);
+                       if (ext_len == -2) {
+                               spprintf(error, 0, "Cannot create a phar archive from a URL like \"%s\".  Phar objects can only be created from local files", fname);
+                       } else {
+                               spprintf(error, 0, "Cannot create phar '%s', file extension (or combination) not recognised", fname);
+                       }
                }
                return FAILURE;
        }
@@ -1903,6 +1907,12 @@ int phar_detect_phar_fname_ext(const char *filename, int filename_len, const cha
        pos = memchr(filename, '/', filename_len);
 
        if (pos && pos != filename) {
+               /* check for url like http:// or phar:// */
+               if (*(pos - 1) == ':' && (pos - filename) < filename_len - 1 && *(pos + 1) == '/') {
+                       *ext_len = -2;
+                       *ext_str = NULL;
+                       return FAILURE;
+               }
                if (zend_hash_exists(&(PHAR_GLOBALS->phar_alias_map), (char *) filename, pos - filename)) {
                        *ext_str = pos;
                        *ext_len = -1;

diff --git a/ext/phar/tests/phar_construct_invalidurl.phpt b/ext/phar/tests/phar_construct_invalidurl.phpt
new file mode 100644 (file)
index 0000000..faac841
--- /dev/null
+++ b/ext/phar/tests/phar_construct_invalidurl.phpt
@@ -0,0 +1,30 @@
+--TEST--
+Phar object passed URL
+--INI--
+default_charset=UTF-8
+--SKIPIF--
+<?php if (!extension_loaded("phar")) die("skip"); ?>
+--FILE--
+<?php
+try {
+       $a = new Phar('http://should.fail.com');
+} catch (UnexpectedValueException $e) {
+       echo $e->getMessage(),"\n";
+}
+try {
+       $a = new Phar('http://');
+} catch (UnexpectedValueException $e) {
+       echo $e->getMessage(),"\n";
+}
+try {
+       $a = new Phar('http:/');
+} catch (UnexpectedValueException $e) {
+       echo $e->getMessage(),"\n";
+}
+?>
+===DONE===
+--EXPECT--
+Cannot create a phar archive from a URL like "http://should.fail.com".  Phar objects can only be created from local files
+Cannot create a phar archive from a URL like "http://".  Phar objects can only be created from local files
+Cannot create phar 'http:/', file extension (or combination) not recognised
+===DONE===
\ No newline at end of file