Commit 8f93fc78 authored by Jeff Trawick's avatar Jeff Trawick
Browse files

Update the hash table of active script pids even on paths where a 

script process wasn't created (storing 0 for the pid in that case).
Otherwise, the remembered pid is that of the last successful script
execution for this hash key.

Prior to this patch, the wrong process could be terminated in
rare circumstances:

- A CGI process with pid 10101 is forked for connection 99.

- After the CGI exits and some time elapses, some other process gets 
  pid 10101. (Connection 99 hasn't handled another CGI request yet.)

- The next time connection 99 has a CGI process, the fork()
  or other early setup fails, so no CGI process is created.

- The remembered pid for connection 99 is still 10101.  It
  gets terminated (subject to permissions).


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@503340 13f79535-47bb-0310-9956-ffa450edef68
parent ff71eb8f

modules/generators/mod_cgid.c

+29 −22
Original line number Diff line number Diff line
@@ -626,6 +626,7 @@ static int cgid_server(void *data) 
        apr_file_t *inout;

        cgid_req_t cgid_req;

        apr_status_t stat;

        void *key;



        apr_pool_clear(ptrans);
@@ -718,6 +719,8 @@ static int cgid_server(void *data) 
             */

            ap_log_error(APLOG_MARK, APLOG_ERR, rc, r->server,

                         "couldn't set child process attributes: %s", r->filename);



            procnew->pid = 0; /* no process to clean up */

        }

        else {

            apr_pool_userdata_set(r, ERRFN_USERDATA_KEY, apr_pool_cleanup_null, ptrans);
@@ -754,9 +757,16 @@ static int cgid_server(void *data) 
                ap_log_error(APLOG_MARK, APLOG_ERR, rc, r->server,

                             "couldn't create child process: %d: %s", rc,

                             apr_filepath_name_get(r->filename));



                procnew->pid = 0; /* no process to clean up */

            }

            else {

                /* We don't want to leak storage for the key, so only allocate

        }



        /* If the script process was created, remember the pid for

         * later cleanup.  If the script process wasn't created, clear

         * out any prior pid with the same key.

         *

         * We don't want to leak storage for the key, so only allocate

         * a key if the key doesn't exist yet in the hash; there are

         * only a limited number of possible keys (one for each

         * possible thread in the server), so we can allocate a copy
@@ -766,7 +776,6 @@ static int cgid_server(void *data) 
         * first time; new key storage isn't needed for replacing the

         * existing value of a key.

         */

                void *key;



        if (apr_hash_get(script_hash, &cgid_req.conn_id, sizeof(cgid_req.conn_id))) {

            key = &cgid_req.conn_id;
@@ -778,8 +787,6 @@ static int cgid_server(void *data) 
        apr_hash_set(script_hash, key, sizeof(cgid_req.conn_id),

                     (void *)((long)procnew->pid));

    }

        }

    }

    return -1;

}