add test for file_get_contents interception, fix file_get_contents interception

diff --git a/ext/phar/func_interceptors.c b/ext/phar/func_interceptors.c
index c814545464c7108733f35a5a09cf0ca1f3cd02e1..b18f052b7f2d2dd1225f63e5aba89a1fd0255312 100644 (file)
--- a/ext/phar/func_interceptors.c
+++ b/ext/phar/func_interceptors.c
@@ -33,7 +33,7 @@ PHAR_FUNC(phar_opendir) /* {{{ */
                return;
        }
 
-       if (!IS_ABSOLUTE_PATH(filename, filename_len)) {
+       if (!IS_ABSOLUTE_PATH(filename, filename_len) && !strstr(filename, "://")) {
                char *arch, *entry, *fname;
                int arch_len, entry_len, fname_len;
                fname = zend_get_executed_filename(TSRMLS_C);
@@ -98,10 +98,11 @@ PHAR_FUNC(phar_file_get_contents) /* {{{ */
        if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS() TSRMLS_CC, "s|br!ll", &filename, &filename_len, &use_include_path, &zcontext, &offset, &maxlen) == FAILURE) {
                return;
        }
-       if (use_include_path || !IS_ABSOLUTE_PATH(filename, filename_len)) {
+       if (use_include_path || (!IS_ABSOLUTE_PATH(filename, filename_len) && !strstr(filename, "://"))) {
                char *arch, *entry, *fname;
                int arch_len, entry_len, fname_len;
                php_stream_context *context = NULL;
+
                fname = zend_get_executed_filename(TSRMLS_C);
 
                if (strncasecmp(fname, "phar://", 7)) {
@@ -116,11 +117,6 @@ PHAR_FUNC(phar_file_get_contents) /* {{{ */
                        entry = filename;
                        /* fopen within phar, if :// is not in the url, then prepend phar://<archive>/ */
                        entry_len = filename_len;
-                       if (strstr(entry, "://")) {
-                               efree(arch);
-                               efree(old);
-                               goto skip_phar;
-                       }
 
                        if (ZEND_NUM_ARGS() == 5 && maxlen < 0) {
                                efree(arch);
@@ -129,41 +125,45 @@ PHAR_FUNC(phar_file_get_contents) /* {{{ */
                                RETURN_FALSE;
                        }
 
-                       if (!IS_ABSOLUTE_PATH(filename, filename_len)) {
-                               /* retrieving a file defaults to within the current directory, so use this if possible */
-                               if (SUCCESS == (zend_hash_find(&(PHAR_GLOBALS->phar_fname_map), arch, arch_len, (void **) &pphar))) {
-                                       if (use_include_path) {
-                                               if (!(entry = phar_find_in_include_path(entry, entry_len, NULL TSRMLS_CC))) {
-                                                       /* this file is not in the phar, use the original path */
-                                                       efree(arch);
-                                                       efree(old);
-                                                       goto skip_phar;
-                                               }
-                                       } else {
-                                               entry = phar_fix_filepath(entry, &entry_len, 1 TSRMLS_CC);
-                                               if (!zend_hash_exists(&((*pphar)->manifest), entry, entry_len)) {
-                                                       /* this file is not in the phar, use the original path */
-                                                       efree(arch);
-                                                       efree(old);
-                                                       efree(entry);
-                                                       goto skip_phar;
-                                               }
-                                       }
-                               }
+                       /* retrieving a file defaults to within the current directory, so use this if possible */
+                       if (FAILURE == (zend_hash_find(&(PHAR_GLOBALS->phar_fname_map), arch, arch_len, (void **) &pphar))) {
+                               efree(arch);
+                               efree(old);
+                               goto skip_phar;
                        }
-                       /* auto-convert to phar:// */
-                       spprintf(&name, 4096, "phar://%s/%s", arch, entry);
-                       if (entry != filename) {
-                               efree(entry);
+                       if (use_include_path) {
+                               if ((entry = phar_find_in_include_path(entry, entry_len, NULL TSRMLS_CC))) {
+                                       name = entry;
+                               } else {
+                                       /* this file is not in the phar, use the original path */
+                                       efree(arch);
+                                       efree(old);
+                                       goto skip_phar;
+                               }
+                       } else {
+                               entry = phar_fix_filepath(estrndup(entry, entry_len), &entry_len, 1 TSRMLS_CC);
+                               if (!zend_hash_exists(&((*pphar)->manifest), entry + 1, entry_len - 1)) {
+                                       /* this file is not in the phar, use the original path */
+                                       efree(arch);
+                                       efree(old);
+                                       efree(entry);
+                                       goto skip_phar;
+                               }
+                               /* auto-convert to phar:// */
+                               spprintf(&name, 4096, "phar://%s%s", arch, entry);
+                               if (entry != filename) {
+                                       efree(entry);
+                               }
                        }
+
                        efree(arch);
+                       efree(old);
                        if (zcontext) {
                                context = php_stream_context_from_zval(zcontext, 0);
                        }
                        stream = php_stream_open_wrapper_ex(name, "rb", 0 | REPORT_ERRORS, NULL, context);
                        efree(name);
 
-
                        if (!stream) {
                                RETURN_FALSE;
                        }
@@ -214,7 +214,7 @@ PHAR_FUNC(phar_fopen) /* {{{ */
        if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS() TSRMLS_CC, "ss|br", &filename, &filename_len, &mode, &mode_len, &use_include_path, &zcontext) == FAILURE) {
                goto skip_phar;
        }
-       if (use_include_path || !IS_ABSOLUTE_PATH(filename, filename_len)) {
+       if (!use_include_path && !IS_ABSOLUTE_PATH(filename, filename_len) && !strstr(filename, "://")) {
                char *arch, *entry, *fname;
                int arch_len, entry_len, fname_len;
                php_stream_context *context = NULL;
@@ -488,7 +488,7 @@ void phar_file_stat(const char *filename, php_stat_len filename_length, int type
                RETURN_FALSE;
        }
 
-       if (!IS_ABSOLUTE_PATH(filename, filename_length)) {
+       if (!IS_ABSOLUTE_PATH(filename, filename_length) && !strstr(filename, "://")) {
                char *arch, *entry, *fname;
                int arch_len, entry_len, fname_len;
                struct stat sb = {0};
@@ -749,7 +749,7 @@ PHAR_FUNC(phar_is_file) /* {{{ */
        if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS() TSRMLS_CC, "s", &filename, &filename_len) == FAILURE) {
                goto skip_phar;
        }
-       if (!IS_ABSOLUTE_PATH(filename, filename_len)) {
+       if (!IS_ABSOLUTE_PATH(filename, filename_len) && !strstr(filename, "://")) {
                char *arch, *entry, *fname;
                int arch_len, entry_len, fname_len, free_filename = 0;
                fname = zend_get_executed_filename(TSRMLS_C);
@@ -804,7 +804,7 @@ PHAR_FUNC(phar_is_link) /* {{{ */
        if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS() TSRMLS_CC, "s", &filename, &filename_len) == FAILURE) {
                goto skip_phar;
        }
-       if (!IS_ABSOLUTE_PATH(filename, filename_len)) {
+       if (!IS_ABSOLUTE_PATH(filename, filename_len) && !strstr(filename, "://")) {
                char *arch, *entry, *fname;
                int arch_len, entry_len, fname_len, free_filename = 0;
                fname = zend_get_executed_filename(TSRMLS_C);

diff --git a/ext/phar/tests/file_get_contents.phpt b/ext/phar/tests/file_get_contents.phpt
new file mode 100644 (file)
index 0000000..fcc9d64
--- /dev/null
+++ b/ext/phar/tests/file_get_contents.phpt
@@ -0,0 +1,30 @@
+--TEST--
+Phar: test file_get_contents() interception
+--SKIPIF--
+<?php if (!extension_loaded("phar")) die("skip");?>
+--INI--
+phar.require_hash=1
+phar.readonly=0
+--FILE--
+<?php
+$fname = dirname(__FILE__) . '/' . basename(__FILE__, '.php') . '.phar.php';
+$a = new Phar($fname);
+$a['index.php'] = '<?php
+echo file_get_contents("dir/file1.txt");
+echo file_get_contents("file1.txt", true);
+?>';
+$a['dir/file1.txt'] = 'hi';
+$a['dir/file2.txt'] = 'hi2';
+$a['dir/file3.txt'] = 'hi3';
+$a->setStub('<?php
+Phar::interceptFileFuncs();
+set_include_path("phar://" . __FILE__ . "/dir" . PATH_SEPARATOR . "phar://" . __FILE__);
+include "index.php";
+__HALT_COMPILER();');
+include $fname;
+?>
+===DONE===
+--CLEAN--
+<?php unlink(dirname(__FILE__) . '/' . basename(__FILE__, '.clean.php') . '.phar.php'); ?>
+--EXPECT--
+hihi===DONE===
\ No newline at end of file