- Fixed bug #49182 (PHP CGI always outputs the shebang line)

diff --git a/main/fopen_wrappers.c b/main/fopen_wrappers.c
index 83a91cc4eefe82824d52763b8349474d600ea6cd..b7cd6dceb23b19f77b1ab5efd1a8709df476cb1f 100644 (file)
--- a/main/fopen_wrappers.c
+++ b/main/fopen_wrappers.c
@@ -334,9 +334,12 @@ static FILE *php_fopen_and_set_opened_path(const char *path, const char *mode, c
  */
 PHPAPI int php_fopen_primary_script(zend_file_handle *file_handle TSRMLS_DC)
 {
+       FILE *fp;
+#ifndef PHP_WIN32
+       struct stat st;
+#endif
        char *path_info, *filename;
        int length;
-       zend_bool orig_display_errors;
 
        filename = SG(request_info).path_translated;
        path_info = SG(request_info).request_uri;
@@ -403,7 +406,7 @@ PHPAPI int php_fopen_primary_script(zend_file_handle *file_handle TSRMLS_DC)
                }
        } /* if doc_root && path_info */
 
-       if(filename) {
+       if (filename) {
                filename = zend_resolve_path(filename, strlen(filename) TSRMLS_CC);
        }
 
@@ -415,20 +418,31 @@ PHPAPI int php_fopen_primary_script(zend_file_handle *file_handle TSRMLS_DC)
                STR_FREE(SG(request_info).path_translated);
                SG(request_info).path_translated = NULL;
                return FAILURE;
-       } else {
-               STR_FREE(SG(request_info).path_translated);
-               SG(request_info).path_translated = filename;
        }
+       fp = VCWD_FOPEN(filename, "rb");
+
+#ifndef PHP_WIN32
+       /* refuse to open anything that is not a regular file */
+       if (fp && (0 > fstat(fileno(fp), &st) || !S_ISREG(st.st_mode))) {
+               fclose(fp);
+               fp = NULL;
+       }
+#endif
 
-       orig_display_errors = PG(display_errors);
-       PG(display_errors) = 0;
-       if (zend_stream_open(filename, file_handle TSRMLS_CC) == FAILURE) {
-               PG(display_errors) = orig_display_errors;
+       if (!fp) {
                STR_FREE(SG(request_info).path_translated);     /* for same reason as above */
                SG(request_info).path_translated = NULL;
                return FAILURE;
        }
-       PG(display_errors) = orig_display_errors;
+
+       file_handle->opened_path = expand_filepath(filename, NULL TSRMLS_CC);
+
+       SG(request_info).path_translated = filename;
+
+       file_handle->filename = SG(request_info).path_translated;
+       file_handle->free_filename = 0;
+       file_handle->handle.fp = fp;
+       file_handle->type = ZEND_HANDLE_FP;
 
        return SUCCESS;
 }

diff --git a/sapi/cgi/cgi_main.c b/sapi/cgi/cgi_main.c
index 230e19f2a2f96f612fcd7e139ef5c0fe45b2ce66..51f3c9229bf2af3138375c7bd3e5456d7210c7ab 100644 (file)
--- a/sapi/cgi/cgi_main.c
+++ b/sapi/cgi/cgi_main.c
@@ -159,6 +159,7 @@ static const opt_struct OPTIONS[] = {
 typedef struct _php_cgi_globals_struct {
        zend_bool rfc2616_headers;
        zend_bool nph;
+       zend_bool check_shebang_line;
        zend_bool fix_pathinfo;
        zend_bool force_redirect;
        zend_bool discard_path;
@@ -1366,6 +1367,7 @@ void fastcgi_cleanup(int signal)
 PHP_INI_BEGIN()
        STD_PHP_INI_ENTRY("cgi.rfc2616_headers",     "0",  PHP_INI_ALL,    OnUpdateBool,   rfc2616_headers, php_cgi_globals_struct, php_cgi_globals)
        STD_PHP_INI_ENTRY("cgi.nph",                 "0",  PHP_INI_ALL,    OnUpdateBool,   nph, php_cgi_globals_struct, php_cgi_globals)
+       STD_PHP_INI_ENTRY("cgi.check_shebang_line",  "1",  PHP_INI_SYSTEM, OnUpdateBool,   check_shebang_line, php_cgi_globals_struct, php_cgi_globals)
        STD_PHP_INI_ENTRY("cgi.force_redirect",      "1",  PHP_INI_SYSTEM, OnUpdateBool,   force_redirect, php_cgi_globals_struct, php_cgi_globals)
        STD_PHP_INI_ENTRY("cgi.redirect_status_env", NULL, PHP_INI_SYSTEM, OnUpdateString, redirect_status_env, php_cgi_globals_struct, php_cgi_globals)
        STD_PHP_INI_ENTRY("cgi.fix_pathinfo",        "1",  PHP_INI_SYSTEM, OnUpdateBool,   fix_pathinfo, php_cgi_globals_struct, php_cgi_globals)
@@ -1382,6 +1384,7 @@ static void php_cgi_globals_ctor(php_cgi_globals_struct *php_cgi_globals TSRMLS_
 {
        php_cgi_globals->rfc2616_headers = 0;
        php_cgi_globals->nph = 0;
+       php_cgi_globals->check_shebang_line = 1;
        php_cgi_globals->force_redirect = 1;
        php_cgi_globals->redirect_status_env = NULL;
        php_cgi_globals->fix_pathinfo = 1;
@@ -1448,6 +1451,7 @@ int main(int argc, char *argv[])
        int exit_status = SUCCESS;
        int cgi = 0, c, i, len;
        zend_file_handle file_handle;
+       int retval = FAILURE;
        char *s;
 
        /* temporary locals */
@@ -2051,6 +2055,26 @@ consult the installation file that came with this distribution, or visit \n\
                                }
                        }
 
+                       if (CGIG(check_shebang_line) && file_handle.handle.fp && (file_handle.handle.fp != stdin)) {
+                               /* #!php support */
+                               c = fgetc(file_handle.handle.fp);
+                               if (c == '#') {
+                                       while (c != '\n' && c != '\r' && c != EOF) {
+                                               c = fgetc(file_handle.handle.fp);       /* skip to end of line */
+                                       }
+                                       /* handle situations where line is terminated by \r\n */
+                                       if (c == '\r') {
+                                               if (fgetc(file_handle.handle.fp) != '\n') {
+                                                       long pos = ftell(file_handle.handle.fp);
+                                                       fseek(file_handle.handle.fp, pos - 1, SEEK_SET);
+                                               }
+                                       }
+                                       CG(start_lineno) = 2;
+                               } else {
+                                       rewind(file_handle.handle.fp);
+                               }
+                       }
+
                        switch (behavior) {
                                case PHP_MODE_STANDARD:
                                        php_execute_script(&file_handle TSRMLS_CC);