sniper          Fri Jun  3 19:11:51 2005 EDT

  Modified files:              
    /php-src/sapi/cgi   cgi_main.c 
  Log:
  cs + ws
  
http://cvs.php.net/diff.php/php-src/sapi/cgi/cgi_main.c?r1=1.264&r2=1.265&ty=u
Index: php-src/sapi/cgi/cgi_main.c
diff -u php-src/sapi/cgi/cgi_main.c:1.264 php-src/sapi/cgi/cgi_main.c:1.265
--- php-src/sapi/cgi/cgi_main.c:1.264   Thu Apr 28 10:23:57 2005
+++ php-src/sapi/cgi/cgi_main.c Fri Jun  3 19:11:51 2005
@@ -20,7 +20,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: cgi_main.c,v 1.264 2005/04/28 14:23:57 sniper Exp $ */
+/* $Id: cgi_main.c,v 1.265 2005/06/03 23:11:51 sniper Exp $ */
 
 #include "php.h"
 #include "php_globals.h"
@@ -76,7 +76,6 @@
 #include "zend_highlight.h"
 #include "zend_indent.h"
 
-
 #include "php_getopt.h"
 
 #if PHP_FASTCGI
@@ -150,7 +149,7 @@
 
 #if ENABLE_PATHINFO_CHECK
 /* true global.  this is retreived once only, even for fastcgi */
-long fix_pathinfo=1;
+long fix_pathinfo = 1;
 #endif
 
 #ifdef PHP_WIN32
@@ -178,7 +177,7 @@
        Bucket *s = *((Bucket **) b);
 
        return strcasecmp(((zend_module_entry *)f->pData)->name,
-                                 ((zend_module_entry *)s->pData)->name);
+                                         ((zend_module_entry 
*)s->pData)->name);
 }
 
 static void print_modules(TSRMLS_D)
@@ -230,8 +229,8 @@
 
 #if PHP_FASTCGI
        if (!FCGX_IsCGI()) {
-               FCGX_Request *request = (FCGX_Request *)SG(server_context);
-               long ret = FCGX_PutStr( str, str_length, request->out );
+               FCGX_Request *request = (FCGX_Request *) SG(server_context);
+               long ret = FCGX_PutStr(str, str_length, request->out);
                if (ret <= 0) {
                        return 0;
                }
@@ -272,7 +271,7 @@
 {
 #if PHP_FASTCGI
        if (!FCGX_IsCGI()) {
-               FCGX_Request *request = (FCGX_Request *)server_context;
+               FCGX_Request *request = (FCGX_Request *) server_context;
                if (
 #ifndef PHP_WIN32
                !parent && 
@@ -283,7 +282,7 @@
                return;
        }
 #endif
-       if (fflush(stdout)==EOF) {
+       if (fflush(stdout) == EOF) {
                php_handle_aborted_connection();
        }
 }
@@ -297,7 +296,7 @@
        zend_llist_position pos;
        long rfc2616_headers = 0, nph = 0;
 
-       if(SG(request_info).no_headers == 1) {
+       if (SG(request_info).no_headers == 1) {
                return  SAPI_HEADER_SENT_SUCCESSFULLY;
        }
        /* Check wheater to send RFC2616 style headers compatible with
@@ -313,7 +312,8 @@
                nph = 0;
        }
 
-       if (nph || SG(sapi_headers).http_response_code != 200) {
+       if (nph || SG(sapi_headers).http_response_code != 200)
+       {
                int len;
 
                if (rfc2616_headers && SG(sapi_headers).http_status_line) {
@@ -331,18 +331,19 @@
                PHPWRITE_H(buf, len);
        }
 
-       if (SG(sapi_headers).send_default_content_type) {
+       if (SG(sapi_headers).send_default_content_type)
+       {
                char *hd;
 
                hd = sapi_get_default_content_type(TSRMLS_C);
-               PHPWRITE_H("Content-type: ", sizeof("Content-type: ")-1);
+               PHPWRITE_H("Content-type: ", sizeof("Content-type: ") - 1);
                PHPWRITE_H(hd, strlen(hd));
                PHPWRITE_H("\r\n", 2);
                efree(hd);
        }
 
        h = zend_llist_get_first_ex(&sapi_headers->headers, &pos);
-    while (h) {
+       while (h) {
                PHPWRITE_H(h->header, h->header_len);
                PHPWRITE_H("\r\n", 2);
                h = zend_llist_get_next_ex(&sapi_headers->headers, &pos);
@@ -360,21 +361,21 @@
        char *pos = buffer;
 #endif
 
-       count_bytes = MIN(count_bytes, 
(uint)SG(request_info).content_length-SG(read_post_bytes));
+       count_bytes = MIN(count_bytes, (uint) SG(request_info).content_length - 
SG(read_post_bytes));
        while (read_bytes < count_bytes) {
 #if PHP_FASTCGI
                if (!FCGX_IsCGI()) {
-                       FCGX_Request *request = (FCGX_Request 
*)SG(server_context);
-                       tmp_read_bytes = FCGX_GetStr( pos, 
count_bytes-read_bytes, request->in );
+                       FCGX_Request *request = (FCGX_Request *) 
SG(server_context);
+                       tmp_read_bytes = FCGX_GetStr(pos, count_bytes - 
read_bytes, request->in);
                        pos += tmp_read_bytes;
                } else {
-                       tmp_read_bytes = read(0, buffer+read_bytes, 
count_bytes-read_bytes);
+                       tmp_read_bytes = read(0, buffer + read_bytes, 
count_bytes - read_bytes);
                }
 #else
-               tmp_read_bytes = read(0, buffer+read_bytes, 
count_bytes-read_bytes);
+               tmp_read_bytes = read(0, buffer + read_bytes, count_bytes - 
read_bytes);
 #endif
 
-               if (tmp_read_bytes<=0) {
+               if (tmp_read_bytes <= 0) {
                        break;
                }
                read_bytes += tmp_read_bytes;
@@ -390,7 +391,7 @@
           of a request.  So we have to do our own lookup to get env
           vars.  This could probably be faster somehow.  */
        if (!FCGX_IsCGI()) {
-               FCGX_Request *request = (FCGX_Request *)SG(server_context);
+               FCGX_Request *request = (FCGX_Request *) SG(server_context);
                return FCGX_GetParam(name,request->envp);
        }
 #endif
@@ -401,20 +402,20 @@
 
 static char *_sapi_cgibin_putenv(char *name, char *value TSRMLS_DC)
 {
-       int len=0;
+       int len = 0;
        char *buf = NULL;
        if (!name) {
                return NULL;
        }
-       len = strlen(name) + (value?strlen(value):0) + sizeof("=") + 2;
-       buf = (char *)malloc(len);
+       len = strlen(name) + (value ? strlen(value) : 0) + sizeof("=") + 2;
+       buf = (char *) malloc(len);
        if (buf == NULL) {
                return getenv(name);
        }
        if (value) {
-               snprintf(buf,len-1,"%s=%s", name, value);
+               snprintf(buf, len - 1, "%s=%s", name, value);
        } else {
-               snprintf(buf,len-1,"%s=", name);
+               snprintf(buf, len - 1, "%s=", name);
        }
 #if PHP_FASTCGI
        /* when php is started by mod_fastcgi, no regular environment
@@ -422,8 +423,8 @@
           of a request.  So we have to do our own lookup to get env
           vars.  This could probably be faster somehow.  */
        if (!FCGX_IsCGI()) {
-               FCGX_Request *request = (FCGX_Request *)SG(server_context);
-               FCGX_PutEnv(request,buf);
+               FCGX_Request *request = (FCGX_Request *) SG(server_context);
+               FCGX_PutEnv(request, buf);
                free(buf);
                return sapi_cgibin_getenv(name,0 TSRMLS_CC);
        }
@@ -439,26 +440,26 @@
 
 static char *sapi_cgi_read_cookies(TSRMLS_D)
 {
-       return sapi_cgibin_getenv((char *)"HTTP_COOKIE",0 TSRMLS_CC);
+       return sapi_cgibin_getenv((char *) "HTTP_COOKIE", 0 TSRMLS_CC);
 }
 
 #if PHP_FASTCGI
 void cgi_php_import_environment_variables(zval *array_ptr TSRMLS_DC)
 {
        if (!FCGX_IsCGI()) {
-               FCGX_Request *request = (FCGX_Request *)SG(server_context);
+               FCGX_Request *request = (FCGX_Request *) SG(server_context);
                char **env, *p, *t;
-               /* turn off magic_quotes while importing environment variables 
*/
                int magic_quotes_gpc = PG(magic_quotes_gpc);
-               PG(magic_quotes_gpc) = 0;
 
+               /* turn off magic_quotes while importing environment variables 
*/
+               PG(magic_quotes_gpc) = 0;
                for (env = request->envp; env != NULL && *env != NULL; env++) {
                        p = strchr(*env, '=');
                        if (!p) {                               /* malformed 
entry? */
                                continue;
                        }
                        t = estrndup(*env, p - *env);
-                       php_register_variable(t, p+1, array_ptr TSRMLS_CC);
+                       php_register_variable(t, p + 1, array_ptr TSRMLS_CC);
                        efree(t);
                }
                PG(magic_quotes_gpc) = magic_quotes_gpc;
@@ -475,10 +476,9 @@
         */
        php_import_environment_variables(track_vars_array TSRMLS_CC);
        /* Build the special-case PHP_SELF variable for the CGI version */
-       php_register_variable("PHP_SELF", (SG(request_info).request_uri ? 
SG(request_info).request_uri:""), track_vars_array TSRMLS_CC);
+       php_register_variable("PHP_SELF", (SG(request_info).request_uri ? 
SG(request_info).request_uri : ""), track_vars_array TSRMLS_CC);
 }
 
-
 static void sapi_cgi_log_message(char *message)
 {
 #if PHP_FASTCGI
@@ -488,10 +488,10 @@
        if (cfg_get_long("fastcgi.logging", &logging) == FAILURE) {
                logging = 1;
        }
-                                                                               
                         
+
        if (!FCGX_IsCGI() && logging) {
-               FCGX_Request *request = (FCGX_Request *)SG(server_context);
-               FCGX_FPrintF( request->err, "%s\n", message );
+               FCGX_Request *request = (FCGX_Request *) SG(server_context);
+               FCGX_FPrintF(request->err, "%s\n", message);
                /* ignore return code */
        } else
 #endif /* PHP_FASTCGI */
@@ -505,7 +505,7 @@
                2. When the first call occurs and the request is not set up, 
flush fails on 
                        FastCGI.
        */
-       if(SG(sapi_started)) {
+       if (SG(sapi_started)) {
                sapi_cgibin_flush(SG(server_context));
        }
        return SUCCESS;
@@ -665,8 +665,8 @@
  */
 static void init_request_info(TSRMLS_D)
 {
-       char *env_script_filename = sapi_cgibin_getenv("SCRIPT_FILENAME",0 
TSRMLS_CC);
-       char *env_path_translated = sapi_cgibin_getenv("PATH_TRANSLATED",0 
TSRMLS_CC);
+       char *env_script_filename = sapi_cgibin_getenv("SCRIPT_FILENAME", 0 
TSRMLS_CC);
+       char *env_path_translated = sapi_cgibin_getenv("PATH_TRANSLATED", 0 
TSRMLS_CC);
        char *script_path_translated = env_script_filename;
 
 #if !DISCARD_PATH
@@ -694,29 +694,29 @@
           of the script will be retreived later via argc/argv */
        if (script_path_translated) {
                const char *auth;
-               char *content_length = sapi_cgibin_getenv("CONTENT_LENGTH",0 
TSRMLS_CC);
-               char *content_type = sapi_cgibin_getenv("CONTENT_TYPE",0 
TSRMLS_CC);
-               char *env_path_info = sapi_cgibin_getenv("PATH_INFO",0 
TSRMLS_CC);
-               char *env_script_name = sapi_cgibin_getenv("SCRIPT_NAME",0 
TSRMLS_CC);
+               char *content_length = sapi_cgibin_getenv("CONTENT_LENGTH", 0 
TSRMLS_CC);
+               char *content_type = sapi_cgibin_getenv("CONTENT_TYPE", 0 
TSRMLS_CC);
+               char *env_path_info = sapi_cgibin_getenv("PATH_INFO", 0 
TSRMLS_CC);
+               char *env_script_name = sapi_cgibin_getenv("SCRIPT_NAME", 0 
TSRMLS_CC);
 #if ENABLE_PATHINFO_CHECK
                struct stat st;
-               char *env_redirect_url = sapi_cgibin_getenv("REDIRECT_URL",0 
TSRMLS_CC);
-               char *env_document_root = sapi_cgibin_getenv("DOCUMENT_ROOT",0 
TSRMLS_CC);
+               char *env_redirect_url = sapi_cgibin_getenv("REDIRECT_URL", 0 
TSRMLS_CC);
+               char *env_document_root = sapi_cgibin_getenv("DOCUMENT_ROOT", 0 
TSRMLS_CC);
 
                if (fix_pathinfo) {
 
                        /* save the originals first for anything we change 
later */
                        if (env_path_translated) {
-                               
_sapi_cgibin_putenv("ORIG_PATH_TRANSLATED",env_path_translated TSRMLS_CC);
+                               _sapi_cgibin_putenv("ORIG_PATH_TRANSLATED", 
env_path_translated TSRMLS_CC);
                        }
                        if (env_path_info) {
-                               
_sapi_cgibin_putenv("ORIG_PATH_INFO",env_path_info TSRMLS_CC);
+                               _sapi_cgibin_putenv("ORIG_PATH_INFO", 
env_path_info TSRMLS_CC);
                        }
                        if (env_script_name) {
-                               
_sapi_cgibin_putenv("ORIG_SCRIPT_NAME",env_script_name TSRMLS_CC);
+                               _sapi_cgibin_putenv("ORIG_SCRIPT_NAME", 
env_script_name TSRMLS_CC);
                        }
                        if (env_script_filename) {
-                               
_sapi_cgibin_putenv("ORIG_SCRIPT_FILENAME",env_script_filename TSRMLS_CC);
+                               _sapi_cgibin_putenv("ORIG_SCRIPT_FILENAME", 
env_script_filename TSRMLS_CC);
                        }
                        if (!env_document_root) {
                                /* ini version of document root */
@@ -726,7 +726,7 @@
                                /* set the document root, this makes a more
                                   consistent env for php scripts */
                                if (env_document_root) {
-                                       env_document_root = 
_sapi_cgibin_putenv("DOCUMENT_ROOT",env_document_root TSRMLS_CC);
+                                       env_document_root = 
_sapi_cgibin_putenv("DOCUMENT_ROOT", env_document_root TSRMLS_CC);
                                        /* fix docroot */
                                        TRANSLATE_SLASHES(env_document_root);
                                }
@@ -740,13 +740,13 @@
                                */
                                script_path_translated = env_path_translated;
                                /* we correct SCRIPT_NAME now in case we don't 
have PATH_INFO */
-                               env_script_name = 
_sapi_cgibin_putenv("SCRIPT_NAME",env_redirect_url TSRMLS_CC);
+                               env_script_name = 
_sapi_cgibin_putenv("SCRIPT_NAME", env_redirect_url TSRMLS_CC);
                        }
 
 #ifdef __riscos__
                        /* Convert path to unix format*/
-                       __riscosify_control|=__RISCOSIFY_DONT_CHECK_DIR;
-                       
script_path_translated=__unixify(script_path_translated,0,NULL,1,0);
+                       __riscosify_control |= __RISCOSIFY_DONT_CHECK_DIR;
+                       script_path_translated = 
__unixify(script_path_translated, 0, NULL, 1, 0);
 #endif
                        
                        /*
@@ -754,14 +754,14 @@
                         * of it by stat'ing back through the '/'
                         * this fixes url's like /info.php/test
                         */
-                       if (script_path_translated && stat( 
script_path_translated, &st ) == -1 ) {
+                       if (script_path_translated && 
stat(script_path_translated, &st) == -1 ) {
                                char *pt = estrdup(script_path_translated);
                                int len = strlen(pt);
                                char *ptr;
 
-                               while( (ptr = strrchr(pt,'/')) || (ptr = 
strrchr(pt,'\\')) ) {
+                               while ((ptr = strrchr(pt, '/')) || (ptr = 
strrchr(pt, '\\'))) {
                                        *ptr = 0;
-                                       if ( stat(pt, &st) == 0 && 
S_ISREG(st.st_mode) ) {
+                                       if (stat(pt, &st) == 0 && 
S_ISREG(st.st_mode)) {
                                                /*
                                                 * okay, we found the base 
script!
                                                 * work out how many chars we 
had to strip off;
@@ -778,22 +778,24 @@
                                                 * out what SCRIPT_NAME should 
be
                                                 */
                                                int slen = len - strlen(pt);
-                                               int pilen = strlen( 
env_path_info );
+                                               int pilen = 
strlen(env_path_info);
                                                char *path_info = env_path_info 
+ pilen - slen;
 
-                                               env_path_info = 
_sapi_cgibin_putenv("PATH_INFO",path_info TSRMLS_CC);
-                                               script_path_translated = 
_sapi_cgibin_putenv("SCRIPT_FILENAME",pt TSRMLS_CC);
+                                               env_path_info = 
_sapi_cgibin_putenv("PATH_INFO", path_info TSRMLS_CC);
+                                               script_path_translated = 
_sapi_cgibin_putenv("SCRIPT_FILENAME", pt TSRMLS_CC);
                                                TRANSLATE_SLASHES(pt);
 
                                                /* figure out docroot
                                                   SCRIPT_FILENAME minus 
SCRIPT_NAME
-                                                  */
+                                               */
 
-                                               if (env_document_root) {
+                                               if (env_document_root)
+                                               {
                                                        int l = 
strlen(env_document_root);
                                                        int path_translated_len 
= 0;
                                                        char *path_translated = 
NULL;
-                                                       if 
(env_document_root[l-1]=='/') {
+                                                       
+                                                       if (env_document_root[l 
- 1] == '/') {
                                                                --l;
                                                        }
 
@@ -803,31 +805,29 @@
                                                         *
                                                         * SCRIPT_NAME is the 
portion of the path beyond docroot
                                                         */
-                                                       env_script_name = 
_sapi_cgibin_putenv("SCRIPT_NAME",pt+l TSRMLS_CC);
+                                                       env_script_name = 
_sapi_cgibin_putenv("SCRIPT_NAME", pt + l TSRMLS_CC);
 
-                                                       /*
-                                                        * PATH_TRANSATED = 
DOCUMENT_ROOT + PATH_INFO
-                                                        */
+                                                       /* PATH_TRANSATED = 
DOCUMENT_ROOT + PATH_INFO */
                                                        path_translated_len = l 
+ strlen(env_path_info) + 2;
-                                                       path_translated = (char 
*)emalloc(path_translated_len);
+                                                       path_translated = (char 
*) emalloc(path_translated_len);
                                                        *path_translated = 0;
-                                                       
strncat(path_translated,env_document_root,l);
-                                                       
strcat(path_translated,env_path_info);
-                                                       env_path_translated = 
_sapi_cgibin_putenv("PATH_TRANSLATED",path_translated TSRMLS_CC);
+                                                       
strncat(path_translated, env_document_root, l);
+                                                       strcat(path_translated, 
env_path_info);
+                                                       env_path_translated = 
_sapi_cgibin_putenv("PATH_TRANSLATED", path_translated TSRMLS_CC);
                                                        efree(path_translated);
-                                               } else if (env_script_name &&
-                                                       
strstr(pt,env_script_name)) {
-                                                       /*
-                                                        * PATH_TRANSATED = 
PATH_TRANSATED - SCRIPT_NAME + PATH_INFO
-                                                        */
-                                                       int ptlen = 
strlen(pt)-strlen(env_script_name);
+                                               } else if (env_script_name && 
+                                                                  strstr(pt, 
env_script_name)
+                                               ) {
+                                                       /* PATH_TRANSATED = 
PATH_TRANSATED - SCRIPT_NAME + PATH_INFO */
+                                                       int ptlen = strlen(pt) 
- strlen(env_script_name);
                                                        int path_translated_len 
= ptlen + strlen(env_path_info) + 2;
                                                        char *path_translated = 
NULL;
-                                                       path_translated = (char 
*)emalloc(path_translated_len);
+
+                                                       path_translated = (char 
*) emalloc(path_translated_len);
                                                        *path_translated = 0;
-                                                       
strncat(path_translated,pt,ptlen);
-                                                       
strcat(path_translated,env_path_info);
-                                                       env_path_translated = 
_sapi_cgibin_putenv("PATH_TRANSLATED",path_translated TSRMLS_CC);
+                                                       
strncat(path_translated, pt, ptlen);
+                                                       strcat(path_translated, 
env_path_info);
+                                                       env_path_translated = 
_sapi_cgibin_putenv("PATH_TRANSLATED", path_translated TSRMLS_CC);
                                                        efree(path_translated);
                                                }
                                                break;
@@ -839,7 +839,7 @@
                                         * a valid path... we will fail, badly. 
of course we would
                                         * have failed anyway... we output 'no 
input file' now.
                                         */
-                                       script_path_translated = 
_sapi_cgibin_putenv("SCRIPT_FILENAME",NULL TSRMLS_CC);
+                                       script_path_translated = 
_sapi_cgibin_putenv("SCRIPT_FILENAME", NULL TSRMLS_CC);
                                        SG(sapi_headers).http_response_code = 
404;
                                }
                                if (pt) {
@@ -847,7 +847,7 @@
                                }
                        } else {
                                /* make sure path_info/translated are empty */
-                               script_path_translated = 
_sapi_cgibin_putenv("SCRIPT_FILENAME",script_path_translated TSRMLS_CC);
+                               script_path_translated = 
_sapi_cgibin_putenv("SCRIPT_FILENAME", script_path_translated TSRMLS_CC);
                                _sapi_cgibin_putenv("PATH_INFO", NULL 
TSRMLS_CC);
                                _sapi_cgibin_putenv("PATH_TRANSLATED", NULL 
TSRMLS_CC);
                        }
@@ -867,16 +867,16 @@
 #if ENABLE_PATHINFO_CHECK
                }
 #endif
-               SG(request_info).request_method = 
sapi_cgibin_getenv("REQUEST_METHOD",0 TSRMLS_CC);
+               SG(request_info).request_method = 
sapi_cgibin_getenv("REQUEST_METHOD", 0 TSRMLS_CC);
                /* FIXME - Work out proto_num here */
-               SG(request_info).query_string = 
sapi_cgibin_getenv("QUERY_STRING",0 TSRMLS_CC);
+               SG(request_info).query_string = 
sapi_cgibin_getenv("QUERY_STRING", 0 TSRMLS_CC);
                /* some server configurations allow '..' to slip through in the
                   translated path.   We'll just refuse to handle such a path. 
*/
                if (script_path_translated && !strstr(script_path_translated, 
"..")) {
                        SG(request_info).path_translated = 
estrdup(script_path_translated);
                }
                SG(request_info).content_type = (content_type ? content_type : 
"" );
-               SG(request_info).content_length = 
(content_length?atoi(content_length):0);
+               SG(request_info).content_length = (content_length ? 
atoi(content_length) : 0);
                
                /* The CGI RFC allows servers to pass on unvalidated 
Authorization data */
                auth = sapi_cgibin_getenv("HTTP_AUTHORIZATION",0 TSRMLS_CC);
@@ -897,7 +897,7 @@
        } else {
                value = "1";
        }
-       zend_alter_ini_entry(name, strlen(name)+1, value, strlen(value), 
PHP_INI_SYSTEM, PHP_INI_STAGE_ACTIVATE);
+       zend_alter_ini_entry(name, strlen(name) + 1, value, strlen(value), 
PHP_INI_SYSTEM, PHP_INI_STAGE_ACTIVATE);
 }
 
 
@@ -922,20 +922,19 @@
  */
 void fastcgi_cleanup(int signal)
 {
-
 #ifdef DEBUG_FASTCGI
-       fprintf( stderr, "FastCGI shutdown, pid %d\n", getpid() );
+       fprintf(stderr, "FastCGI shutdown, pid %d\n", getpid());
 #endif
 
 #ifndef PHP_WIN32
-       sigaction( SIGTERM, &old_term, 0 );
+       sigaction(SIGTERM, &old_term, 0);
 
        /* Kill all the processes in our process group */
-       kill( -pgroup, SIGTERM );
+       kill(-pgroup, SIGTERM);
 #endif
 
        /* We should exit at this point, but MacOSX doesn't seem to */
-       exit( 0 );
+       exit(0);
 }
 #endif
 
@@ -949,11 +948,11 @@
        int retval = FAILURE;
        char *s;
 /* temporary locals */
-       int behavior=PHP_MODE_STANDARD;
-       int no_headers=0;
-       int orig_optind=php_optind;
-       char *orig_optarg=php_optarg;
-       char *script_file=NULL;
+       int behavior = PHP_MODE_STANDARD;
+       int no_headers = 0;
+       int orig_optind = php_optind;
+       char *orig_optarg = php_optarg;
+       char *script_file = NULL;
        zend_llist global_vars;
 #if FORCE_CGI_REDIRECT
        long force_redirect = 1;
@@ -981,7 +980,7 @@
 #ifdef PHP_WIN32
        long impersonate = 0;
 #else
-    int status = 0;
+       int status = 0;
 #endif
 #endif /* PHP_FASTCGI */
 
@@ -991,9 +990,8 @@
        {
                char szMessage [256];
                wsprintf (szMessage, "Please attach a debugger to the process 
0x%X [%d] (%s) and click OK",
-                         GetCurrentProcessId(),GetCurrentProcessId(), argv[0]);
-               MessageBox(NULL, szMessage, "CGI Debug Time!",
-                         MB_OK|MB_SERVICE_NOTIFICATION);
+                         GetCurrentProcessId(), GetCurrentProcessId(), 
argv[0]);
+               MessageBox(NULL, szMessage, "CGI Debug Time!", 
MB_OK|MB_SERVICE_NOTIFICATION);
        }
 #endif
 
@@ -1015,10 +1013,10 @@
        sapi_startup(&cgi_sapi_module);
 
 #ifdef PHP_WIN32
-       _fmode = _O_BINARY;                     /*sets default for file streams 
to binary */
-       setmode(_fileno(stdin), O_BINARY);              /* make the stdio mode 
be binary */
-       setmode(_fileno(stdout), O_BINARY);             /* make the stdio mode 
be binary */
-       setmode(_fileno(stderr), O_BINARY);             /* make the stdio mode 
be binary */
+       _fmode = _O_BINARY; /* sets default for file streams to binary */
+       setmode(_fileno(stdin),  O_BINARY);     /* make the stdio mode be 
binary */
+       setmode(_fileno(stdout), O_BINARY);     /* make the stdio mode be 
binary */
+       setmode(_fileno(stderr), O_BINARY);     /* make the stdio mode be 
binary */
 #endif
 
 #if PHP_FASTCGI
@@ -1026,17 +1024,18 @@
 #endif
        /* Make sure we detect we are a cgi - a bit redundancy here,
           but the default case is that we have to check only the first one. */
-       if (getenv("SERVER_SOFTWARE")
-               || getenv("SERVER_NAME")
-               || getenv("GATEWAY_INTERFACE")
-               || getenv("REQUEST_METHOD")) {
+       if (getenv("SERVER_SOFTWARE") ||
+               getenv("SERVER_NAME") ||
+               getenv("GATEWAY_INTERFACE") ||
+               getenv("REQUEST_METHOD")
+       ) {
                cgi = 1;
        }
 #if PHP_FASTCGI
        }
 #endif
 
-       while ((c=php_getopt(argc, argv, OPTIONS, &php_optarg, &php_optind, 
0))!=-1) {
+       while ((c = php_getopt(argc, argv, OPTIONS, &php_optarg, &php_optind, 
0)) != -1) {
                switch (c) {
                        case 'c':
                                cgi_sapi_module.php_ini_path_override = 
strdup(php_optarg);
@@ -1084,12 +1083,12 @@
 #if FORCE_CGI_REDIRECT
        /* check force_cgi after startup, so we have proper output */
        if (cfg_get_long("cgi.force_redirect", &force_redirect) == FAILURE) {
-        force_redirect = 1;
+               force_redirect = 1;
        }
        if (cgi && force_redirect) {
-        if (cfg_get_string("cgi.redirect_status_env", &redirect_status_env) == 
FAILURE) {
-            redirect_status_env = NULL;
-        }
+               if (cfg_get_string("cgi.redirect_status_env", 
&redirect_status_env) == FAILURE) {
+                       redirect_status_env = NULL;
+               }
                /* Apache will generate REDIRECT_STATUS,
                 * Netscape and redirect.so will generate HTTP_REDIRECT_STATUS.
                 * redirect.so and installation instructions available from
@@ -1099,7 +1098,7 @@
                if (!getenv("REDIRECT_STATUS")
                        && !getenv ("HTTP_REDIRECT_STATUS")
                        /* this is to allow a different env var to be configured
-                           in case some server does something different than 
above */
+                          in case some server does something different than 
above */
                        && (!redirect_status_env || 
!getenv(redirect_status_env))
                        ) {
                        SG(sapi_headers).http_response_code = 400;
@@ -1114,19 +1113,18 @@
 <a href=\"http://php.net/install.windows\";>the manual page</a>.</p>\n");
 
 #if defined(ZTS) && !defined(PHP_DEBUG)
-               /* XXX we're crashing here in msvc6 debug builds at
-                  php_message_handler_for_zend:839 because
-                  SG(request_info).path_translated is an invalid pointer.
-                  It still happens even though I set it to null, so something
-                  weird is going on.
-               */
-               tsrm_shutdown();
+                       /* XXX we're crashing here in msvc6 debug builds at
+                          php_message_handler_for_zend:839 because
+                          SG(request_info).path_translated is an invalid 
pointer.
+                          It still happens even though I set it to null, so 
something
+                          weird is going on.
+                       */
+                       tsrm_shutdown();
 #endif
-
                        return FAILURE;
                }
        }
-#endif                                                 /* FORCE_CGI_REDIRECT */
+#endif /* FORCE_CGI_REDIRECT */
 
 #if ENABLE_PATHINFO_CHECK
        if (cfg_get_long("cgi.fix_pathinfo", &fix_pathinfo) == FAILURE) {
@@ -1159,10 +1157,10 @@
                } else {
                        fcgi_fd = FCGX_OpenSocket(bindpath, 128);
                }
-               if( fcgi_fd < 0 ) {
+               if (fcgi_fd < 0) {
                        fprintf(stderr, "Couldn't create FastCGI listen socket 
on port %s\n", bindpath);
 #ifdef ZTS
-               tsrm_shutdown();
+                       tsrm_shutdown();
 #endif
                        return FAILURE;
                }
@@ -1171,11 +1169,10 @@
 #endif
        if (fastcgi) {
                /* How many times to run PHP scripts before dying */
-               if( getenv( "PHP_FCGI_MAX_REQUESTS" )) {
-                       max_requests = atoi( getenv( "PHP_FCGI_MAX_REQUESTS" ));
-                       if( !max_requests ) {
-                               fprintf( stderr,
-                                        "PHP_FCGI_MAX_REQUESTS is not valid\n" 
);
+               if (getenv("PHP_FCGI_MAX_REQUESTS")) {
+                       max_requests = atoi(getenv("PHP_FCGI_MAX_REQUESTS"));
+                       if (!max_requests) {
+                               fprintf(stderr, "PHP_FCGI_MAX_REQUESTS is not 
valid\n");
                                return FAILURE;
                        }
                }
@@ -1186,20 +1183,19 @@
 
                /* library is already initialized, now init our request */
                FCGX_Init();
-               FCGX_InitRequest( &request, fcgi_fd, 0 );
+               FCGX_InitRequest(&request, fcgi_fd, 0);
 
 #ifndef PHP_WIN32
        /* Pre-fork, if required */
-       if( getenv( "PHP_FCGI_CHILDREN" )) {
-               children = atoi( getenv( "PHP_FCGI_CHILDREN" ));
-               if( !children ) {
-                       fprintf( stderr,
-                                "PHP_FCGI_CHILDREN is not valid\n" );
+       if (getenv("PHP_FCGI_CHILDREN")) {
+               children = atoi(getenv("PHP_FCGI_CHILDREN"));
+               if (!children) {
+                       fprintf(stderr, "PHP_FCGI_CHILDREN is not valid\n");
                        return FAILURE;
                }
        }
 
-       if( children ) {
+       if (children) {
                int running = 0;
                pid_t pid;
 
@@ -1207,27 +1203,26 @@
                setsid();
                pgroup = getpgrp();
 #ifdef DEBUG_FASTCGI
-               fprintf( stderr, "Process group %d\n", pgroup );
+               fprintf(stderr, "Process group %d\n", pgroup);
 #endif
 
                /* Set up handler to kill children upon exit */
                act.sa_flags = 0;
                act.sa_handler = fastcgi_cleanup;
-               if( sigaction( SIGTERM, &act, &old_term ) ||
-                   sigaction( SIGINT, &act, &old_int ) ||
-                   sigaction( SIGQUIT, &act, &old_quit )) {
-                       perror( "Can't set signals" );
-                       exit( 1 );
+               if (sigaction(SIGTERM, &act, &old_term) ||
+                   sigaction(SIGINT,  &act, &old_int) ||
+                   sigaction(SIGQUIT, &act, &old_quit)) {
+                       perror("Can't set signals");
+                       exit(1);
                }
 
-               while( parent ) {
+               while (parent) {
                        do {
 #ifdef DEBUG_FASTCGI
-                               fprintf( stderr, "Forking, %d running\n",
-                                        running );
+                               fprintf(stderr, "Forking, %d running\n", 
running);
 #endif
                                pid = fork();
-                               switch( pid ) {
+                               switch (pid) {
                                case 0:
                                        /* One of the children.
                                         * Make sure we don't go round the
@@ -1236,27 +1231,26 @@
                                        parent = 0;
 
                                        /* don't catch our signals */
-                                       sigaction( SIGTERM, &old_term, 0 );
-                                       sigaction( SIGQUIT, &old_quit, 0 );
-                                       sigaction( SIGINT, &old_int, 0 );
+                                       sigaction(SIGTERM, &old_term, 0);
+                                       sigaction(SIGQUIT, &old_quit, 0);
+                                       sigaction(SIGINT,  &old_int,  0);
                                        break;
                                case -1:
-                                       perror( "php (pre-forking)" );
-                                       exit( 1 );
+                                       perror("php (pre-forking)");
+                                       exit(1);
                                        break;
                                default:
                                        /* Fine */
                                        running++;
                                        break;
                                }
-                       } while( parent && ( running < children ));
+                       } while (parent && (running < children));
 
-                       if( parent ) {
+                       if (parent) {
 #ifdef DEBUG_FASTCGI
-                               fprintf( stderr, "Wait for kids, pid %d\n",
-                                        getpid() );
+                               fprintf(stderr, "Wait for kids, pid %d\n", 
getpid());
 #endif
-                               wait( &status );
+                               wait(&status);
                                running--;
                        }
                }
@@ -1267,7 +1261,7 @@
 #endif /* FASTCGI */
 
        zend_first_try {
-               while ((c=php_getopt(argc, argv, OPTIONS, &php_optarg, 
&php_optind, 1))!=-1) {
+               while ((c = php_getopt(argc, argv, OPTIONS, &php_optarg, 
&php_optind, 1)) != -1) {
                        switch (c) {
                                case 'h':
                                case '?':
@@ -1287,7 +1281,6 @@
 #if PHP_FASTCGI
                /* start of FAST CGI loop */
                /* Initialise FastCGI request structure */
-
 #ifdef PHP_WIN32
                /* attempt to set security impersonation for fastcgi
                   will only happen on NT based OS, others will ignore it. */
@@ -1298,9 +1291,7 @@
                        if (impersonate) OS_SetImpersonate();
                }
 #endif
-
-               while (!fastcgi
-                       || FCGX_Accept_r( &request ) >= 0) {
+               while (!fastcgi || FCGX_Accept_r(&request) >= 0) {
 #endif
 
 #if PHP_FASTCGI
@@ -1308,19 +1299,15 @@
 #else
                SG(server_context) = (void *) 1; /* avoid server_context==NULL 
checks */
 #endif
-
                init_request_info(TSRMLS_C);
-
                zend_llist_init(&global_vars, sizeof(char *), NULL, 0);
-
                CG(interactive) = 0;
 
                if (!cgi
 #if PHP_FASTCGI
                        && !fastcgi
 #endif
-                       ) {
-
+               ) {
                        if (cgi_sapi_module.php_ini_path_override && 
cgi_sapi_module.php_ini_ignore) {
                                no_headers = 1;
                                php_output_startup();
@@ -1354,8 +1341,8 @@
                                                script_file = 
estrdup(php_optarg);
                                                no_headers = 1;
                                                /* arguments after the file are 
considered script args */
-                                               SG(request_info).argc = argc - 
(php_optind-1);
-                                               SG(request_info).argv = 
&argv[php_optind-1];
+                                               SG(request_info).argc = argc - 
(php_optind - 1);
+                                               SG(request_info).argv = 
&argv[php_optind - 1];
                                                break;
 
                                case 'g': /* define global variables on command 
line */
@@ -1382,7 +1369,7 @@
 
                                case 'l': /* syntax check mode */
                                                no_headers = 1;
-                                               behavior=PHP_MODE_LINT;
+                                               behavior = PHP_MODE_LINT;
                                                break;
 
                                case 'm': /* list compiled in modules */
@@ -1409,7 +1396,7 @@
                                                break;
 
                                case 's': /* generate highlighted HTML from 
source */
-                                               behavior=PHP_MODE_HIGHLIGHT;
+                                               behavior = PHP_MODE_HIGHLIGHT;
                                                break;
 
                                case 'v': /* show php version & quit */
@@ -1432,7 +1419,7 @@
                                                break;
 
                                case 'w':
-                                               behavior=PHP_MODE_STRIP;
+                                               behavior = PHP_MODE_STRIP;
                                                break;
 
                                case 'z': /* load extension file */
@@ -1514,7 +1501,7 @@
 
                /* request startup only after we've done all we can to
                   get path_translated */
-               if (php_request_startup(TSRMLS_C)==FAILURE) {
+               if (php_request_startup(TSRMLS_C) == FAILURE) {
 #if PHP_FASTCGI
                        if (fastcgi) {
                                FCGX_Finish_r(&request);
@@ -1655,7 +1642,7 @@
                        if (!fastcgi) break;
                        /* only fastcgi will get here */
                        requests++;
-                       if(max_requests && (requests == max_requests)) {
+                       if (max_requests && (requests == max_requests)) {
                                FCGX_Finish_r(&request);
 #ifndef PHP_WIN32
                                if (bindpath) {
@@ -1683,7 +1670,7 @@
 #endif
 
 #if PHP_WIN32 && ZEND_DEBUG && 0
-       _CrtDumpMemoryLeaks( );
+       _CrtDumpMemoryLeaks();
 #endif
 
        return exit_status;

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

Reply via email to