]> granicus.if.org Git - apache/blobdiff - modules/generators/mod_cgid.c
very minor tweaks:
[apache] / modules / generators / mod_cgid.c
index e8b1bc537f27d9ee13ed89557f135743f2f8ff7d..c279b30e8fa2cb7f99ea684ccad50c9de17462e5 100644 (file)
@@ -1,7 +1,7 @@
 /* ====================================================================
  * The Apache Software License, Version 1.1
  *
- * Copyright (c) 2000 The Apache Software Foundation.  All rights
+ * Copyright (c) 2000-2001 The Apache Software Foundation.  All rights
  * reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * they fail. 
  */ 
 
-
-
-#define CORE_PRIVATE 
-
 #include "apr_lib.h"
 #include "apr_strings.h"
 #include "apr_general.h"
 #include "apr_file_io.h"
 #include "apr_portable.h"
+#include "apr_buckets.h"
+#include "apr_optional.h"
+#include "apr_signal.h"
+
+#define APR_WANT_STRFUNC
+#include "apr_want.h"
+
+#if APR_HAVE_SYS_SOCKET_H
+#include <sys/socket.h>
+#endif
+#if APR_HAVE_UNISTD_H
+#include <unistd.h>
+#endif
+#if APR_HAVE_SYS_TYPES_H
+#include <sys/types.h>
+#endif
+
+#define CORE_PRIVATE 
+
+#include "util_filter.h"
 #include "httpd.h" 
 #include "http_config.h" 
 #include "http_request.h" 
 #include "http_main.h" 
 #include "http_log.h" 
 #include "util_script.h" 
-#include "http_conf_globals.h" 
-#include "buff.h" 
 #include "ap_mpm.h"
-#include "ap_iol.h"
 #include "unixd.h"
+#include "mod_suexec.h"
+#include "../filters/mod_include.h"
+
+#include "mod_core.h"
+
+
+/* ### should be tossed in favor of APR */
 #include <sys/stat.h>
-#ifdef HAVE_SYS_SOCKET_H
-#include <sys/socket.h>
-#endif
-#ifdef HAVE_UNISTD_H
-#include <unistd.h>
-#endif
-#ifdef HAVE_STRINGS_H
-#include <strings.h>
-#endif
 #include <sys/un.h> /* for sockaddr_un */
-#include <sys/types.h>
 
-module MODULE_VAR_EXPORT cgid_module; 
 
-static void cgid_init(ap_pool_t *p, ap_pool_t *plog, ap_pool_t *ptemp, server_rec *main_server); 
-static int once_through = 0; 
+module AP_MODULE_DECLARE_DATA cgid_module; 
+
+static int cgid_init(apr_pool_t *p, apr_pool_t *plog, apr_pool_t *ptemp, server_rec *main_server); 
+static int handle_exec(include_ctx_t *ctx, apr_bucket_brigade **bb, request_rec *r,
+                       ap_filter_t *f, apr_bucket *head_ptr, apr_bucket **inserted_head);
+
+static APR_OPTIONAL_FN_TYPE(ap_register_include_handler) *cgid_pfn_reg_with_ssi;
+static APR_OPTIONAL_FN_TYPE(ap_ssi_get_tag_and_value) *cgid_pfn_gtv;
+static APR_OPTIONAL_FN_TYPE(ap_ssi_parse_string) *cgid_pfn_ps;
 
-static ap_pool_t *pcgi; 
+static apr_pool_t *pcgi; 
+static int total_modules = 0;
 
 /* KLUDGE --- for back-combatibility, we don't have to check Execcgid 
  * in ScriptAliased directories, which means we need to know if this 
@@ -119,7 +136,7 @@ static ap_pool_t *pcgi;
 
 static int is_scriptaliased(request_rec *r) 
 { 
-    const char *t = ap_table_get(r->notes, "alias-forced-type"); 
+    const char *t = apr_table_get(r->notes, "alias-forced-type"); 
     return t && (!strcasecmp(t, "cgi-script")); 
 } 
 
@@ -129,7 +146,8 @@ static int is_scriptaliased(request_rec *r)
 #define DEFAULT_BUFBYTES 1024 
 #define DEFAULT_SOCKET "logs/cgisock"
 
-#define SHELL_PATH "/bin/sh"
+#define CGI_REQ 1
+#define SSI_REQ 2
 
 /* DEFAULT_CGID_LISTENBACKLOG controls the max depth on the unix socket's
  * pending connection queue.  If a bunch of cgi requests arrive at about
@@ -147,9 +165,6 @@ typedef struct {
     const char *logname; 
     long logbytes; 
     int bufbytes; 
-    BUFF *bin; 
-    BUFF *bout; 
-    BUFF *berror; 
 } cgid_server_conf; 
 
 /* If a request includes query info in the URL (stuff after "?"), and
@@ -165,7 +180,7 @@ typedef struct {
  * are handled in create_argv.
  *
  */
-static char **create_argv(ap_pool_t *p, char *path, char *user, char *group,
+static char **create_argv(apr_pool_t *p, char *path, char *user, char *group,
                           char *av0, const char *args)
 {
     int x, numwords;
@@ -184,7 +199,7 @@ static char **create_argv(ap_pool_t *p, char *path, char *user, char *group,
     if (numwords > APACHE_ARG_MAX - 5) {
         numwords = APACHE_ARG_MAX - 5;  /* Truncate args to prevent overrun */
     }
-    av = (char **) ap_palloc(p, (numwords + 5) * sizeof(char *));
+    av = (char **) apr_pcalloc(p, (numwords + 5) * sizeof(char *));
 
     if (path) {
         av[idx++] = path;
@@ -194,158 +209,66 @@ static char **create_argv(ap_pool_t *p, char *path, char *user, char *group,
     }
     if (group) {
         av[idx++] = group;
-     }
+    }
 
-    av[idx++] = av0;
+    av[idx++] = apr_pstrdup(p, av0);
 
     for (x = 1; x <= numwords; x++) {
         w = ap_getword_nulls(p, &args, '+');
-        ap_unescape_url(w);
-        av[idx++] = ap_escape_shell_cmd(p, w);
+        if (strcmp(w, "")) {
+            ap_unescape_url(w);
+            av[idx++] = ap_escape_shell_cmd(p, w);
+        }
     }
     av[idx] = NULL;
     return av;
 }
 
-static int call_exec(request_rec *r, char *argv0, char **env, int shellcmd)
+#if APR_HAS_OTHER_CHILD
+static void cgid_maint(int reason, void *data, apr_wait_t status)
 {
-    int pid = 0;
-    int errfileno = STDERR_FILENO;
-    /* the fd on r->server->error_log is closed, but we need somewhere to            
-     * put the error messages from the log_* functions. So, we use stderr,
-     * since that is better than allowing errors to go unnoticed. 
-     */
-    ap_put_os_file(&r->server->error_log, &errfileno, r->pool);
-    /* TODO: reimplement suexec */
-#if 0
-    if (ap_suexec_enabled
-        && ((r->server->server_uid != ap_user_id)
-            || (r->server->server_gid != ap_group_id)
-            || (!strncmp("/~", r->uri, 2)))) {
-
-        char *execuser, *grpname;
-        struct passwd *pw;
-        struct group *gr;
-
-        if (!strncmp("/~", r->uri, 2)) {
-            gid_t user_gid;
-            char *username = ap_pstrdup(r->pool, r->uri + 2);
-            char *pos = strchr(username, '/');
-
-            if (pos) {
-                *pos = '\0';
-            }
-
-            if ((pw = getpwnam(username)) == NULL) {
-                ap_log_rerror(APLOG_MARK, APLOG_ERR, r,
-                             "getpwnam: invalid username %s", username);
-                return (pid);
-            }
-            execuser = ap_pstrcat(r->pool, "~", pw->pw_name, NULL);
-            user_gid = pw->pw_gid;
-
-            if ((gr = getgrgid(user_gid)) == NULL) {
-                if ((grpname = ap_palloc(r->pool, 16)) == NULL) {
-                    return (pid);
-                }
-                else {
-                    ap_snprintf(grpname, 16, "%ld", (long) user_gid);
-                }
-            }
-            else {
-                grpname = gr->gr_name;
-            }
-        }
-        else {
-            if ((pw = getpwuid(r->server->server_uid)) == NULL) {
-                ap_log_rerror(APLOG_MARK, APLOG_ERR, r,
-                             "getpwuid: invalid userid %ld",
-                             (long) r->server->server_uid);
-                return (pid);
-            }
-            execuser = ap_pstrdup(r->pool, pw->pw_name);
-
-            if ((gr = getgrgid(r->server->server_gid)) == NULL) {
-                ap_log_rerror(APLOG_MARK, APLOG_ERR, r,
-                             "getgrgid: invalid groupid %ld",
-                             (long) r->server->server_gid);
-                return (pid);
-            }
-            grpname = gr->gr_name;
-        }
-
-        if (shellcmd) {
-            execle(SUEXEC_BIN, SUEXEC_BIN, execuser, grpname, argv0,
-                   NULL, env);
-        }
-
-        else if ((!r->args) || (!r->args[0]) || strchr(r->args, '=')) {
-            execle(SUEXEC_BIN, SUEXEC_BIN, execuser, grpname, argv0,
-                   NULL, env);
-        }
-
-        else {
-            execve(SUEXEC_BIN,
-                   create_argv(r->pool, SUEXEC_BIN, execuser, grpname,
-                               argv0, r->args),
-                   env);
-        }
-    }
-    else {
-#endif
-        if (shellcmd) {
-            execle(SHELL_PATH, SHELL_PATH, "-c", argv0, NULL, env);
-        }
-
-        else if ((!r->args) || (!r->args[0]) || strchr(r->args, '=')) {
-            execle(r->filename, argv0, NULL, env);
-        }
+    pid_t *sd = data;
 
-        else {
-            execve(r->filename,
-                   create_argv(r->pool, NULL, NULL, NULL, argv0, r->args),
-                   env);
-        }
-#if 0
-    }
-#endif
-    return (pid);
-}
-
-static void cgid_maint(int reason, void *data, ap_wait_t status)
-{
-#if APR_HAS_OTHER_CHILD
-    int *sd = data;
     switch (reason) {
         case APR_OC_REASON_DEATH:
+            /* don't do anything; server is stopping or restarting */
+            break;
         case APR_OC_REASON_LOST:
-            /* stop gap to make sure everything else works.  In the end,
-             * we'll just restart the cgid server. */
-            ap_destroy_pool(pcgi);
-            kill(getppid(), SIGWINCH);
+            /* it would be better to restart just the cgid child
+             * process but for now we'll gracefully restart the entire 
+             * server by sending AP_SIG_GRACEFUL to ourself, the httpd 
+             * parent process
+             */
+            kill(getpid(), AP_SIG_GRACEFUL);
             break;
         case APR_OC_REASON_RESTART:
+            apr_proc_other_child_unregister(data);
+            break;
         case APR_OC_REASON_UNREGISTER:
-            ap_destroy_pool(pcgi);
+            /* we get here when pcgi is cleaned up; pcgi gets cleaned
+             * up when pconf gets cleaned up
+             */
             kill(*sd, SIGHUP);
             break;
     }
-#endif
 }
+#endif
 
-static void get_req(int fd, request_rec *r, char **filename, char **argv0, char ***env
+static void get_req(int fd, request_rec *r, char **argv0, char ***env, int *req_type
 { 
     int i, len, j; 
     unsigned char *data; 
     char **environ; 
     core_dir_config *temp_core; 
     void **dconf; 
+    module *suexec_mod = ap_find_linked_module("mod_suexec.c");
 
-    r->server = ap_pcalloc(r->pool, sizeof(server_rec)); 
+    r->server = apr_pcalloc(r->pool, sizeof(server_rec)); 
 
+    read(fd, req_type, sizeof(int));
     read(fd, &j, sizeof(int)); 
     read(fd, &len, sizeof(int)); 
-    data = ap_pcalloc(r->pool, len + 1); /* get a cleared byte for final '\0' */
+    data = apr_pcalloc(r->pool, len + 1); /* get a cleared byte for final '\0' */
     i = read(fd, data, len); 
 
     r->filename = ap_getword(r->pool, (const char **)&data, '\n'); 
@@ -353,7 +276,7 @@ static void get_req(int fd, request_rec *r, char **filename, char **argv0, char
 
     r->uri = ap_getword(r->pool, (const char **)&data, '\n'); 
     
-    environ = ap_pcalloc(r->pool, (j + 2) *sizeof(char *)); 
+    environ = apr_pcalloc(r->pool, (j + 2) *sizeof(char *)); 
     i = 0; 
     for (i = 0; i < j; i++) { 
         environ[i] = ap_getword(r->pool, (const char **)&data, '\n'); 
@@ -361,20 +284,32 @@ static void get_req(int fd, request_rec *r, char **filename, char **argv0, char
     *env = environ; 
     r->args = ap_getword(r->pool, (const char **)&data, '\n'); 
   
-    read(fd, &r->server->server_uid, sizeof(uid_t)); 
-    read(fd, &r->server->server_gid, sizeof(gid_t)); 
-
     read(fd, &i, sizeof(int)); 
      
     /* add 1, so that if i == 0, we still malloc something. */ 
-    dconf = (void **)malloc(sizeof(void *) * i + 1); 
 
-    temp_core = (core_dir_config *)malloc(sizeof(core_module)); 
+    dconf = (void **) apr_pcalloc(r->pool, sizeof(void *) * (total_modules + DYNAMIC_MODULE_LIMIT));
+
+    temp_core = (core_dir_config *)apr_palloc(r->pool, sizeof(core_module)); 
+
+    dconf[i] = (void *)temp_core; 
+
+    if (suexec_mod) {
+        suexec_config_t *suexec_cfg = apr_pcalloc(r->pool, sizeof(*suexec_cfg));
+
+        read(fd, &i, sizeof(int));
+        read(fd, &suexec_cfg->ugid.uid, sizeof(uid_t));
+        read(fd, &suexec_cfg->ugid.gid, sizeof(gid_t));
+        read(fd, &suexec_cfg->active, sizeof(int));
+        dconf[i] = (void *)suexec_cfg;
+    }
+
+    r->per_dir_config = (ap_conf_vector_t *)dconf; 
 #if 0
 #ifdef RLIMIT_CPU 
     read(fd, &j, sizeof(int)); 
     if (j) { 
-        temp_core->limit_cpu = (struct rlimit *)malloc (sizeof(struct rlimit)); 
+        temp_core->limit_cpu = (struct rlimit *)apr_palloc (sizeof(struct rlimit)); 
         read(fd, temp_core->limit_cpu, sizeof(struct rlimit)); 
     } 
     else { 
@@ -385,7 +320,7 @@ static void get_req(int fd, request_rec *r, char **filename, char **argv0, char
 #if defined (RLIMIT_DATA) || defined(RLIMIT_VMEM) || defined(RLIMIT_AS) 
     read(fd, &j, sizeof(int)); 
     if (j) { 
-        temp_core->limit_mem = (struct rlimit *)malloc (sizeof(struct rlimit)); 
+        temp_core->limit_mem = (struct rlimit *)apr_palloc(r->pool, sizeof(struct rlimit)); 
         read(fd, temp_core->limit_mem, sizeof(struct rlimit)); 
     } 
     else { 
@@ -396,7 +331,7 @@ static void get_req(int fd, request_rec *r, char **filename, char **argv0, char
 #ifdef RLIMIT_NPROC 
     read(fd, &j, sizeof(int)); 
     if (j) { 
-        temp_core->limit_nproc = (struct rlimit *)malloc (sizeof(struct rlimit)); 
+        temp_core->limit_nproc = (struct rlimit *)apr_palloc(r->pool, sizeof(struct rlimit)); 
         read(fd, temp_core->limit_nproc, sizeof(struct rlimit)); 
     } 
     else { 
@@ -404,55 +339,71 @@ static void get_req(int fd, request_rec *r, char **filename, char **argv0, char
     } 
 #endif 
 #endif
-    dconf[i] = (void *)temp_core; 
-    r->per_dir_config = dconf; 
+    /* For right now, just make the notes table.  At some point we will need
+     * to actually fill this out, but for now we just don't want suexec to
+     * seg fault.
+     */
+    r->notes = apr_table_make(r->pool, 1);
 } 
 
 
 
-static void send_req(int fd, request_rec *r, char *argv0, char **env) 
+static void send_req(int fd, request_rec *r, char *argv0, char **env, int req_type
 { 
-    int len; 
+    int len, r_type = req_type
     int i = 0; 
     char *data; 
+    module *suexec_mod = ap_find_linked_module("mod_suexec.c");
 
-    data = ap_pstrcat(r->pool, r->filename, "\n", argv0, "\n", r->uri, "\n", 
+    data = apr_pstrcat(r->pool, r->filename, "\n", argv0, "\n", r->uri, "\n", 
                      NULL); 
 
     for (i =0; env[i]; i++) { 
         continue; 
     } 
 
+    /* Write the request type (SSI "exec cmd" or cgi). */
+    if (write(fd, &r_type, sizeof(int)) < 0) {
+        ap_log_rerror(APLOG_MARK, APLOG_ERR, errno, r,
+                     "write to cgi daemon process");
+    }
+
+    /* Write the number of entries in the environment. */
     if (write(fd, &i, sizeof(int)) < 0) {
         ap_log_rerror(APLOG_MARK, APLOG_ERR, errno, r, 
                      "write to cgi daemon process"); 
         }     
 
     for (i = 0; env[i]; i++) { 
-        data = ap_pstrcat(r->pool, data, env[i], "\n", NULL); 
+        data = apr_pstrcat(r->pool, data, env[i], "\n", NULL); 
     } 
-    data = ap_pstrcat(r->pool, data, r->args, NULL); 
+    data = apr_pstrcat(r->pool, data, r->args, NULL); 
     len = strlen(data); 
+    /* Write the length of the concatenated env string. */
     if (write(fd, &len, sizeof(int)) < 0) { 
         ap_log_rerror(APLOG_MARK, APLOG_ERR, errno, r, 
                      "write to cgi daemon process"); 
-        }     
+    }
+    /* Write the concatted env string. */     
     if (write(fd, data, len) < 0) {
         ap_log_rerror(APLOG_MARK, APLOG_ERR, errno, r, 
                      "write to cgi daemon process"); 
-        }     
-    if (write(fd, &r->server->server_uid, sizeof(uid_t)) < 0) {
-        ap_log_rerror(APLOG_MARK, APLOG_ERR, errno, r, 
-                     "write to cgi daemon process"); 
-        }     
-    if (write(fd, &r->server->server_gid, sizeof(gid_t)) < 0) { 
-        ap_log_rerror(APLOG_MARK, APLOG_ERR, errno, r, 
-                     "write to cgi daemon process"); 
-        }     
+    }
+    /* Write module_index id value. */     
     if (write(fd, &core_module.module_index, sizeof(int)) < 0) { 
         ap_log_rerror(APLOG_MARK, APLOG_ERR, errno, r, 
                      "write to cgi daemon process"); 
-        }     
+    }     
+    if (suexec_mod) {
+        suexec_config_t *suexec_cfg = ap_get_module_config(r->per_dir_config,
+                                                           suexec_mod);
+
+        write(fd, &suexec_mod->module_index, sizeof(int));
+        write(fd, &suexec_cfg->ugid.uid, sizeof(uid_t));
+        write(fd, &suexec_cfg->ugid.gid, sizeof(gid_t));
+        write(fd, &suexec_cfg->active, sizeof(int));
+    }
+
 #if 0
 #ifdef RLIMIT_CPU 
     if (conf->limit_cpu) { 
@@ -492,38 +443,21 @@ static void send_req(int fd, request_rec *r, char *argv0, char **env)
 #endif 
 } 
 
-static int cgid_server_child(int sd) 
-{ 
-    char *argv0; 
-    char *filename; 
-    char **env; 
-    ap_pool_t *p; 
-    request_rec *r; 
-
-    ap_create_pool(&p, pcgi); 
-    r = ap_pcalloc(p, sizeof(request_rec)); 
-    r->pool = p; 
-    dup2(sd, STDIN_FILENO); 
-    dup2(sd, STDOUT_FILENO); 
-    get_req(sd, r, &filename, &argv0, &env); 
-    call_exec(r, argv0, env, 0); 
-    exit(-1);   /* We should NEVER get here */
-} 
-
 static int cgid_server(void *data) 
 { 
     struct sockaddr_un unix_addr;
-    int pid; 
-    int sd, sd2, len, rc;
-    int errfile;
+    int sd, sd2, rc, req_type;
     mode_t omask;
+    apr_socklen_t len;
+    apr_pool_t *ptrans;
     server_rec *main_server = data;
-    cgid_server_conf *sconf = (cgid_server_conf *)ap_get_module_config( 
-                       main_server->module_config, &cgid_module); 
+    cgid_server_conf *sconf = ap_get_module_config(main_server->module_config,
+                                                   &cgid_module); 
+
+    apr_pool_create(&ptrans, pcgi); 
 
-    ap_signal(SIGCHLD, SIG_IGN); 
-    if (unlink(sconf->sockname) < 0 &&
-        errno != ENOENT) {
+    apr_signal(SIGCHLD, SIG_IGN); 
+    if (unlink(sconf->sockname) < 0 && errno != ENOENT) {
         ap_log_error(APLOG_MARK, APLOG_ERR, errno, main_server,
                      "Couldn't unlink unix domain socket %s",
                      sconf->sockname);
@@ -566,82 +500,166 @@ static int cgid_server(void *data)
     }
     
     unixd_setup_child(); /* if running as root, switch to configured user/group */
-
     while (1) {
+        int errfileno = STDERR_FILENO;
+        char *argv0; 
+        char **env; 
+        const char * const *argv; 
+        apr_int32_t in_pipe;
+        apr_int32_t out_pipe;
+        apr_int32_t err_pipe;
+        apr_cmdtype_e cmd_type;
+        request_rec *r;
+        apr_procattr_t *procattr = NULL;
+        apr_proc_t *procnew = NULL;
+        apr_file_t *inout;
+
+        apr_pool_clear(ptrans);
+
         len = sizeof(unix_addr);
         sd2 = accept(sd, (struct sockaddr *)&unix_addr, &len);
         if (sd2 < 0) {
-            ap_log_error(APLOG_MARK, APLOG_ERR, errno, (server_rec *)data,
-                         "Error accepting on cgid socket.");
+            if (errno != EINTR) {
+                ap_log_error(APLOG_MARK, APLOG_ERR, errno, 
+                             (server_rec *)data,
+                             "Error accepting on cgid socket");
+            }
             continue;
         }
        
-        if ((pid = fork()) > 0) {
+        r = apr_pcalloc(ptrans, sizeof(request_rec)); 
+        procnew = apr_pcalloc(ptrans, sizeof(*procnew));
+        r->pool = ptrans; 
+        get_req(sd2, r, &argv0, &env, &req_type); 
+        apr_os_file_put(&r->server->error_log, &errfileno, r->pool);
+        apr_os_file_put(&inout, &sd2, r->pool);
+
+        if (req_type == SSI_REQ) {
+            in_pipe  = APR_NO_PIPE;
+            out_pipe = APR_FULL_BLOCK;
+            err_pipe = APR_NO_PIPE;
+            cmd_type = APR_SHELLCMD;
+        }
+        else {
+            in_pipe  = APR_CHILD_BLOCK;
+            out_pipe = APR_CHILD_BLOCK;
+            err_pipe = APR_CHILD_BLOCK;
+            cmd_type = APR_PROGRAM;
+        }
+
+        if (((rc = apr_procattr_create(&procattr, ptrans)) != APR_SUCCESS) ||
+            ((req_type == CGI_REQ) && 
+             (((rc = apr_procattr_io_set(procattr,
+                                        in_pipe,
+                                        out_pipe,
+                                        err_pipe)) != APR_SUCCESS) ||
+              /* XXX apr_procattr_child_*_set() is creating an unnecessary 
+               * pipe between this process and the child being created...
+               * It is cleaned up with the temporary pool for this request.
+               */
+              ((rc = apr_procattr_child_err_set(procattr, r->server->error_log, NULL)) != APR_SUCCESS) ||
+              ((rc = apr_procattr_child_in_set(procattr, inout, NULL)) != APR_SUCCESS))) ||
+            ((rc = apr_procattr_child_out_set(procattr, inout, NULL)) != APR_SUCCESS) ||
+            ((rc = apr_procattr_dir_set(procattr,
+                                  ap_make_dirstr_parent(r->pool, r->filename))) != APR_SUCCESS) ||
+            ((rc = apr_procattr_cmdtype_set(procattr, cmd_type)) != APR_SUCCESS)) {
+            /* Something bad happened, tell the world. */
+            ap_log_rerror(APLOG_MARK, APLOG_ERR, rc, r,
+                      "couldn't set child process attributes: %s", r->filename);
+        }
+        else {
+            argv = (const char * const *)create_argv(r->pool, NULL, NULL, NULL, argv0, r->args);
+
+           /* We want to close sd2 for the new CGI process too.
+            * If it is left open it'll make ap_pass_brigade() block
+            * waiting for EOF if CGI forked something running long.
+            * close(sd2) here should be okay, as CGI channel
+            * is already dup()ed by apr_procattr_child_{in,out}_set()
+            * above.
+            */
             close(sd2);
-        } 
-        else if (pid == 0) { 
-            /* setup the STDERR here, because I have all the info
-             * for it.  I'll do the STDIN and STDOUT later, but I can't
-             * do STDERR as easily.
-             */
-            if (sconf->logname) {
-                dup2(open(sconf->logname, O_WRONLY), STDERR_FILENO);
-            }
-            else {
-                ap_get_os_file(&errfile, main_server->error_log);
-                dup2(errfile, STDERR_FILENO);
+
+            rc = ap_os_create_privileged_process(r, procnew, argv0, argv, 
+                                                 (const char * const *)env, 
+                                                 procattr, ptrans);
+
+            if (rc != APR_SUCCESS) {
+                /* Bad things happened. Everyone should have cleaned up. */
+                ap_log_rerror(APLOG_MARK, APLOG_ERR, rc, r,
+                        "couldn't create child process: %d: %s", rc, r->filename);
             }
-            cgid_server_child(sd2); 
-        } 
-        else { 
-            ap_log_error(APLOG_MARK, APLOG_ERR, errno, (server_rec *)data, 
-                         "Couldn't fork cgi script"); 
-        } 
+        }
     } 
     return -1; 
 } 
 
-static void cgid_init(ap_pool_t *p, ap_pool_t *plog, ap_pool_t *ptemp, server_rec *main_server) 
+static int cgid_init(apr_pool_t *p, apr_pool_t *plog, apr_pool_t *ptemp, 
+                      server_rec *main_server) 
 { 
     pid_t pid; 
-    ap_proc_t *procnew;
+    apr_proc_t *procnew;
+    void *data;
+    int first_time = 0;
+    const char *userdata_key = "cgid_init";
+    module **m;
+
+    apr_pool_userdata_get(&data, userdata_key, main_server->process->pool);
+    if (!data) {
+        first_time = 1;
+        apr_pool_userdata_set((const void *)1, userdata_key,
+                         apr_pool_cleanup_null, main_server->process->pool);
+    }
 
-    if (once_through > 0) { 
-        ap_create_pool(&pcgi, p); 
+    if (!first_time) {
+        total_modules = 0;
+        for (m = ap_preloaded_modules; *m != NULL; m++)
+            total_modules++;
 
         if ((pid = fork()) < 0) {
             ap_log_error(APLOG_MARK, APLOG_ERR, errno, main_server, 
                          "Couldn't spawn cgid daemon process"); 
+            /* XXX should we return a failure here ? */
         }
         else if (pid == 0) {
+            apr_pool_create(&pcgi, p); 
             cgid_server(main_server);
             exit(-1);
         } 
-        procnew = ap_pcalloc(p, sizeof(*procnew));        
+        procnew = apr_pcalloc(p, sizeof(*procnew));
         procnew->pid = pid;
         procnew->err = procnew->in = procnew->out = NULL;
-        ap_note_subprocess(p, procnew, kill_after_timeout);
+        apr_pool_note_subprocess(p, procnew, kill_after_timeout);
 #if APR_HAS_OTHER_CHILD
-        ap_register_other_child(procnew, cgid_maint, NULL, NULL, p);
+        apr_proc_other_child_register(procnew, cgid_maint, &procnew->pid, NULL, p);
 #endif
-    } 
-    else once_through++; 
+
+        cgid_pfn_reg_with_ssi = APR_RETRIEVE_OPTIONAL_FN(ap_register_include_handler);
+        cgid_pfn_gtv          = APR_RETRIEVE_OPTIONAL_FN(ap_ssi_get_tag_and_value);
+        cgid_pfn_ps           = APR_RETRIEVE_OPTIONAL_FN(ap_ssi_parse_string);
+
+        if ((cgid_pfn_reg_with_ssi) && (cgid_pfn_gtv) && (cgid_pfn_ps)) {
+            /* Required by mod_include filter. This is how mod_cgid registers
+             *   with mod_include to provide processing of the exec directive.
+             */
+            cgid_pfn_reg_with_ssi("exec", handle_exec);
+        }
+    }
+    return OK;
 } 
 
-static void *create_cgid_config(ap_pool_t *p, server_rec *s) 
+static void *create_cgid_config(apr_pool_t *p, server_rec *s) 
 { 
     cgid_server_conf *c = 
-    (cgid_server_conf *) ap_pcalloc(p, sizeof(cgid_server_conf)); 
+    (cgid_server_conf *) apr_pcalloc(p, sizeof(cgid_server_conf)); 
 
     c->logname = NULL; 
     c->logbytes = DEFAULT_LOGBYTES; 
     c->bufbytes = DEFAULT_BUFBYTES; 
     c->sockname = ap_server_root_relative(p, DEFAULT_SOCKET); 
-    c->bin = c->bout = c->berror = NULL; 
     return c; 
 } 
 
-static void *merge_cgid_config(ap_pool_t *p, void *basev, void *overridesv) 
+static void *merge_cgid_config(apr_pool_t *p, void *basev, void *overridesv) 
 { 
     cgid_server_conf *base = (cgid_server_conf *) basev, *overrides = (cgid_server_conf *) overridesv; 
 
@@ -651,18 +669,18 @@ static void *merge_cgid_config(ap_pool_t *p, void *basev, void *overridesv)
 static const char *set_scriptlog(cmd_parms *cmd, void *dummy, const char *arg) 
 { 
     server_rec *s = cmd->server; 
-    cgid_server_conf *conf = 
-    (cgid_server_conf *) ap_get_module_config(s->module_config, &cgid_module); 
+    cgid_server_conf *conf = ap_get_module_config(s->module_config,
+                                                  &cgid_module); 
 
-    conf->logname = arg; 
+    conf->logname = ap_server_root_relative(cmd->pool, arg);
     return NULL; 
 } 
 
 static const char *set_scriptlog_length(cmd_parms *cmd, void *dummy, const char *arg) 
 { 
     server_rec *s = cmd->server; 
-    cgid_server_conf *conf = 
-    (cgid_server_conf *) ap_get_module_config(s->module_config, &cgid_module); 
+    cgid_server_conf *conf = ap_get_module_config(s->module_config,
+                                                  &cgid_module); 
 
     conf->logbytes = atol(arg); 
     return NULL; 
@@ -671,8 +689,8 @@ static const char *set_scriptlog_length(cmd_parms *cmd, void *dummy, const char
 static const char *set_scriptlog_buffer(cmd_parms *cmd, void *dummy, const char *arg) 
 { 
     server_rec *s = cmd->server; 
-    cgid_server_conf *conf = 
-    (cgid_server_conf *) ap_get_module_config(s->module_config, &cgid_module); 
+    cgid_server_conf *conf = ap_get_module_config(s->module_config,
+                                                  &cgid_module); 
 
     conf->bufbytes = atoi(arg); 
     return NULL; 
@@ -681,8 +699,8 @@ static const char *set_scriptlog_buffer(cmd_parms *cmd, void *dummy, const char
 static const char *set_script_socket(cmd_parms *cmd, void *dummy, const char *arg) 
 { 
     server_rec *s = cmd->server; 
-    cgid_server_conf *conf = 
-    (cgid_server_conf *) ap_get_module_config(s->module_config, &cgid_module); 
+    cgid_server_conf *conf = ap_get_module_config(s->module_config,
+                                                  &cgid_module); 
 
     conf->sockname = ap_server_root_relative(cmd->pool, arg); 
     return NULL; 
@@ -703,117 +721,125 @@ static const command_rec cgid_cmds[] =
 }; 
 
 static int log_scripterror(request_rec *r, cgid_server_conf * conf, int ret, 
-                           int show_errno, char *error) 
+                           apr_status_t rv, char *error) 
 { 
-    ap_file_t *f = NULL; 
+    apr_file_t *f = NULL; 
     struct stat finfo; 
-    char time_str[AP_CTIME_LEN];
+    char time_str[APR_CTIME_LEN];
+    int log_flags = rv ? APLOG_ERR : APLOG_NOERRNO | APLOG_ERR;
 
-    ap_log_rerror(APLOG_MARK, show_errno|APLOG_ERR, errno, r, 
+    ap_log_rerror(APLOG_MARK, log_flags, rv, r, 
                 "%s: %s", error, r->filename); 
 
+    /* XXX Very expensive mainline case! Open, then getfileinfo! */
     if (!conf->logname || 
-        ((stat(ap_server_root_relative(r->pool, conf->logname), &finfo) == 0) 
+        ((stat(conf->logname, &finfo) == 0) 
          && (finfo.st_size > conf->logbytes)) || 
-         (ap_open(&f, ap_server_root_relative(r->pool, conf->logname),
+         (apr_file_open(&f, conf->logname,
                   APR_APPEND|APR_WRITE|APR_CREATE, APR_OS_DEFAULT, r->pool) != APR_SUCCESS)) { 
         return ret; 
     } 
 
     /* "%% [Wed Jun 19 10:53:21 1996] GET /cgid-bin/printenv HTTP/1.0" */ 
-    ap_ctime(time_str, ap_now());
-    ap_fprintf(f, "%%%% [%s] %s %s%s%s %s\n", time_str, r->method, r->uri, 
+    apr_ctime(time_str, apr_time_now());
+    apr_file_printf(f, "%%%% [%s] %s %s%s%s %s\n", time_str, r->method, r->uri, 
             r->args ? "?" : "", r->args ? r->args : "", r->protocol); 
     /* "%% 500 /usr/local/apache/cgid-bin */ 
-    ap_fprintf(f, "%%%% %d %s\n", ret, r->filename); 
+    apr_file_printf(f, "%%%% %d %s\n", ret, r->filename); 
 
-    ap_fprintf(f, "%%error\n%s\n", error); 
+    apr_file_printf(f, "%%error\n%s\n", error); 
 
-    ap_close(f); 
+    apr_file_close(f); 
     return ret; 
 } 
 
 static int log_script(request_rec *r, cgid_server_conf * conf, int ret, 
-                  char *dbuf, const char *sbuf, BUFF *script_in, BUFF *script_err) 
+                  char *dbuf, const char *sbuf, apr_file_t *script_in, apr_file_t *script_err) 
 { 
-    ap_array_header_t *hdrs_arr = ap_table_elts(r->headers_in); 
-    ap_table_entry_t *hdrs = (ap_table_entry_t *) hdrs_arr->elts; 
+    const apr_array_header_t *hdrs_arr = apr_table_elts(r->headers_in); 
+    const apr_table_entry_t *hdrs = (apr_table_entry_t *) hdrs_arr->elts; 
     char argsbuffer[HUGE_STRING_LEN]; 
-    ap_file_t *f = NULL; 
+    apr_file_t *f = NULL; 
     int i; 
     struct stat finfo; 
-    char time_str[AP_CTIME_LEN];
+    char time_str[APR_CTIME_LEN];
 
+    /* XXX Very expensive mainline case! Open, then getfileinfo! */
     if (!conf->logname || 
-        ((stat(ap_server_root_relative(r->pool, conf->logname), &finfo) == 0) 
+        ((stat(conf->logname, &finfo) == 0) 
          && (finfo.st_size > conf->logbytes)) || 
-         (ap_open(&f, ap_server_root_relative(r->pool, conf->logname)
+         (apr_file_open(&f, conf->logname
                   APR_APPEND|APR_WRITE|APR_CREATE, APR_OS_DEFAULT, r->pool) != APR_SUCCESS)) { 
         /* Soak up script output */ 
-        while (ap_bgets(argsbuffer, HUGE_STRING_LEN, script_in) > 0) 
+        while (apr_file_gets(argsbuffer, HUGE_STRING_LEN, 
+                             script_in) == APR_SUCCESS) 
             continue; 
         if (script_err) {
-            while (ap_bgets(argsbuffer, HUGE_STRING_LEN, script_err) > 0) 
+            while (apr_file_gets(argsbuffer, HUGE_STRING_LEN, 
+                                 script_err) == APR_SUCCESS) 
                 continue; 
         }
         return ret; 
     } 
 
     /* "%% [Wed Jun 19 10:53:21 1996] GET /cgid-bin/printenv HTTP/1.0" */ 
-    ap_ctime(time_str, ap_now());
-    ap_fprintf(f, "%%%% [%s] %s %s%s%s %s\n", time_str, r->method, r->uri, 
+    apr_ctime(time_str, apr_time_now());
+    apr_file_printf(f, "%%%% [%s] %s %s%s%s %s\n", time_str, r->method, r->uri, 
             r->args ? "?" : "", r->args ? r->args : "", r->protocol); 
     /* "%% 500 /usr/local/apache/cgid-bin" */ 
-    ap_fprintf(f, "%%%% %d %s\n", ret, r->filename); 
+    apr_file_printf(f, "%%%% %d %s\n", ret, r->filename); 
 
-    ap_puts("%request\n", f); 
+    apr_file_puts("%request\n", f); 
     for (i = 0; i < hdrs_arr->nelts; ++i) { 
         if (!hdrs[i].key) 
             continue; 
-        ap_fprintf(f, "%s: %s\n", hdrs[i].key, hdrs[i].val); 
+        apr_file_printf(f, "%s: %s\n", hdrs[i].key, hdrs[i].val); 
     } 
     if ((r->method_number == M_POST || r->method_number == M_PUT) 
         && *dbuf) { 
-        ap_fprintf(f, "\n%s\n", dbuf); 
+        apr_file_printf(f, "\n%s\n", dbuf); 
     } 
 
-    ap_puts("%response\n", f); 
-    hdrs_arr = ap_table_elts(r->err_headers_out); 
-    hdrs = (ap_table_entry_t *) hdrs_arr->elts; 
+    apr_file_puts("%response\n", f); 
+    hdrs_arr = apr_table_elts(r->err_headers_out); 
+    hdrs = (const apr_table_entry_t *) hdrs_arr->elts; 
 
     for (i = 0; i < hdrs_arr->nelts; ++i) { 
         if (!hdrs[i].key) 
             continue; 
-        ap_fprintf(f, "%s: %s\n", hdrs[i].key, hdrs[i].val); 
+        apr_file_printf(f, "%s: %s\n", hdrs[i].key, hdrs[i].val); 
     } 
 
     if (sbuf && *sbuf) 
-        ap_fprintf(f, "%s\n", sbuf); 
-
-    if (ap_bgets(argsbuffer, HUGE_STRING_LEN, script_in) > 0) { 
-        ap_puts("%stdout\n", f); 
-        ap_puts(argsbuffer, f); 
-        while (ap_bgets(argsbuffer, HUGE_STRING_LEN, script_in) > 0) 
-            ap_puts(argsbuffer, f); 
-        ap_puts("\n", f); 
+        apr_file_printf(f, "%s\n", sbuf); 
+
+    if (apr_file_gets(argsbuffer, HUGE_STRING_LEN, script_in) == APR_SUCCESS) { 
+        apr_file_puts("%stdout\n", f); 
+        apr_file_puts(argsbuffer, f); 
+        while (apr_file_gets(argsbuffer, HUGE_STRING_LEN, 
+                             script_in) == APR_SUCCESS) 
+            apr_file_puts(argsbuffer, f); 
+        apr_file_puts("\n", f); 
     } 
 
     if (script_err) {
-        if (ap_bgets(argsbuffer, HUGE_STRING_LEN, script_err) > 0) { 
-            ap_puts("%stderr\n", f); 
-            ap_puts(argsbuffer, f); 
-            while (ap_bgets(argsbuffer, HUGE_STRING_LEN, script_err) > 0) 
-                ap_puts(argsbuffer, f); 
-            ap_puts("\n", f); 
+        if (apr_file_gets(argsbuffer, HUGE_STRING_LEN, 
+                          script_err) == APR_SUCCESS) { 
+            apr_file_puts("%stderr\n", f); 
+            apr_file_puts(argsbuffer, f); 
+            while (apr_file_gets(argsbuffer, HUGE_STRING_LEN, 
+                                 script_err) == APR_SUCCESS) 
+                apr_file_puts(argsbuffer, f); 
+            apr_file_puts("\n", f); 
         } 
     }
 
-    ap_bclose(script_in); 
+    apr_file_close(script_in); 
     if (script_err) {
-        ap_bclose(script_err); 
+        apr_file_close(script_err); 
     }
 
-    ap_close(f); 
+    apr_file_close(f); 
     return ret; 
 } 
 
@@ -827,26 +853,30 @@ static int cgid_handler(request_rec *r)
 { 
     int retval, nph, dbpos = 0; 
     char *argv0, *dbuf = NULL; 
-    BUFF *script = NULL; 
+    apr_bucket_brigade *bb;
+    apr_bucket *b;
     char argsbuffer[HUGE_STRING_LEN]; 
-    void *sconf = r->server->module_config; 
-    cgid_server_conf *conf = (cgid_server_conf *) ap_get_module_config(sconf, &cgid_module); 
-    int is_included = !strcmp(r->protocol, "INCLUDED"); 
+    cgid_server_conf *conf;
+    int is_included;
     int sd;
     char **env; 
     struct sockaddr_un unix_addr;
-    ap_socket_t *tempsock = NULL;
-    int nbytes;
-    ap_iol *iol;
-    script = ap_bcreate(r->pool, B_RDWR); 
+    apr_file_t *tempsock;
+    apr_size_t nbytes;
+
+    if(strcmp(r->handler,CGI_MAGIC_TYPE) && strcmp(r->handler,"cgi-script"))
+       return DECLINED;
 
     if (r->method_number == M_OPTIONS) { 
         /* 99 out of 100 cgid scripts, this is all they support */ 
-        r->allowed |= (1 << M_GET); 
-        r->allowed |= (1 << M_POST); 
+        r->allowed |= (AP_METHOD_BIT << M_GET); 
+        r->allowed |= (AP_METHOD_BIT << M_POST); 
         return DECLINED; 
     } 
 
+    conf = ap_get_module_config(r->server->module_config, &cgid_module); 
+    is_included = !strcmp(r->protocol, "INCLUDED"); 
+
     if ((argv0 = strrchr(r->filename, '/')) != NULL)
         argv0++;
     else
@@ -860,39 +890,27 @@ static int cgid_handler(request_rec *r)
         argv0 = r->filename; 
 
     if (!(ap_allow_options(r) & OPT_EXECCGI) && !is_scriptaliased(r)) 
-        return log_scripterror(r, conf, HTTP_FORBIDDEN, APLOG_NOERRNO
+        return log_scripterror(r, conf, HTTP_FORBIDDEN, 0
                                "Options ExecCGI is off in this directory"); 
     if (nph && is_included) 
-        return log_scripterror(r, conf, HTTP_FORBIDDEN, APLOG_NOERRNO
+        return log_scripterror(r, conf, HTTP_FORBIDDEN, 0
                                "attempt to include NPH CGI script"); 
 
-#if defined(OS2) || defined(WIN32) 
-    /* Allow for cgid files without the .EXE extension on them under OS/2 */ 
-    if (r->finfo.st_mode == 0) { 
-        struct stat statbuf; 
-        char *newfile; 
-
-        newfile = ap_pstrcat(r->pool, r->filename, ".EXE", NULL); 
-
-        if ((stat(newfile, &statbuf) != 0) || (!S_ISREG(statbuf.st_mode))) { 
-            return log_scripterror(r, conf, HTTP_NOT_FOUND, 0, 
-                                   "script not found or unable to stat"); 
-        } else { 
-            r->filename = newfile; 
-        } 
-    } 
+#if defined(OS2) || defined(WIN32)
+#error mod_cgid does not work on this platform.  If you teach it to, look 
+#error at mod_cgi.c for required code in this path.
 #else 
-    if (r->finfo.protection == 0) 
-        return log_scripterror(r, conf, HTTP_NOT_FOUND, APLOG_NOERRNO
+    if (r->finfo.filetype == 0) 
+        return log_scripterror(r, conf, HTTP_NOT_FOUND, 0
                                "script not found or unable to stat"); 
 #endif 
     if (r->finfo.filetype == APR_DIR) 
-        return log_scripterror(r, conf, HTTP_FORBIDDEN, APLOG_NOERRNO
+        return log_scripterror(r, conf, HTTP_FORBIDDEN, 0
                                "attempt to invoke directory as script"); 
 /*
     if (!ap_suexec_enabled) { 
         if (!ap_can_exec(&r->finfo)) 
-            return log_scripterror(r, conf, HTTP_FORBIDDEN, APLOG_NOERRNO
+            return log_scripterror(r, conf, HTTP_FORBIDDEN, 0
                                    "file permissions deny server execution"); 
     } 
 */
@@ -901,7 +919,7 @@ static int cgid_handler(request_rec *r)
     env = ap_create_environment(r->pool, r->subprocess_env); 
 
     if ((sd = socket(AF_UNIX, SOCK_STREAM, 0)) < 0) {
-            return log_scripterror(r, conf, HTTP_NOT_FOUND, 0
+            return log_scripterror(r, conf, HTTP_INTERNAL_SERVER_ERROR, errno
                                    "unable to create socket to cgi daemon");
     } 
     memset(&unix_addr, 0, sizeof(unix_addr));
@@ -909,17 +927,16 @@ static int cgid_handler(request_rec *r)
     strcpy(unix_addr.sun_path, conf->sockname);
 
     if (connect(sd, (struct sockaddr *)&unix_addr, sizeof(unix_addr)) < 0) {
-            return log_scripterror(r, conf, HTTP_NOT_FOUND, 0
+            return log_scripterror(r, conf, HTTP_INTERNAL_SERVER_ERROR, errno
                                    "unable to connect to cgi daemon");
     } 
 
-    send_req(sd, r, argv0, env); 
-
-    ap_put_os_sock(&tempsock, &sd, pcgi);
+    send_req(sd, r, argv0, env, CGI_REQ); 
 
-    iol = ap_iol_attach_socket(pcgi, tempsock);
-
-    ap_bpush_iol(script, iol); 
+    /* We are putting the tempsock variable into a file so that we can use
+     * a pipe bucket to send the data to the client.
+     */
+    apr_os_file_put(&tempsock, &sd, r->pool);
 
     if ((retval = ap_setup_client_block(r, REQUEST_CHUNKED_ERROR))) 
         return retval; 
@@ -937,12 +954,10 @@ static int cgid_handler(request_rec *r)
         int dbsize, len_read; 
 
         if (conf->logname) { 
-            dbuf = ap_pcalloc(r->pool, conf->bufbytes + 1); 
+            dbuf = apr_pcalloc(r->pool, conf->bufbytes + 1); 
             dbpos = 0; 
         } 
 
-
-
         while ((len_read = 
                 ap_get_client_block(r, argsbuffer, HUGE_STRING_LEN)) > 0) { 
             if (conf->logname) { 
@@ -955,7 +970,8 @@ static int cgid_handler(request_rec *r)
                 memcpy(dbuf + dbpos, argsbuffer, dbsize); 
                 dbpos += dbsize; 
             } 
-            ap_bwrite(script, argsbuffer, len_read, &nbytes);
+            nbytes = len_read;
+            apr_file_write(tempsock, argsbuffer, &nbytes);
             if (nbytes < len_read) { 
                 /* silly script stopped reading, soak up remaining message */ 
                 while (ap_get_client_block(r, argsbuffer, HUGE_STRING_LEN) > 0) { 
@@ -964,40 +980,43 @@ static int cgid_handler(request_rec *r)
                 break; 
             } 
         } 
-
-        ap_bflush(script); 
-
     } 
+    /* we're done writing, or maybe we didn't write at all;
+     * force EOF on child's stdin so that the cgi detects end (or
+     * absence) of data
+     */
+    shutdown(sd, 1);
 
     /* Handle script return... */ 
-    if (script && !nph) { 
+    if (!nph) { 
         const char *location; 
         char sbuf[MAX_STRING_LEN]; 
         int ret; 
 
-        if ((ret = ap_scan_script_header_err_buff(r, script, sbuf))) { 
-            return log_script(r, conf, ret, dbuf, sbuf, script, NULL); 
+        if ((ret = ap_scan_script_header_err(r, tempsock, sbuf))) { 
+            return log_script(r, conf, ret, dbuf, sbuf, tempsock, NULL); 
         } 
 
-        location = ap_table_get(r->headers_out, "Location"); 
+        location = apr_table_get(r->headers_out, "Location"); 
 
         if (location && location[0] == '/' && r->status == 200) { 
 
             /* Soak up all the script output */ 
-            while (ap_bgets(argsbuffer, HUGE_STRING_LEN, script) > 0) { 
+            while (apr_file_gets(argsbuffer, HUGE_STRING_LEN, 
+                                 tempsock) == APR_SUCCESS) { 
                 continue; 
             } 
             /* This redirect needs to be a GET no matter what the original 
              * method was. 
              */ 
-            r->method = ap_pstrdup(r->pool, "GET"); 
+            r->method = apr_pstrdup(r->pool, "GET"); 
             r->method_number = M_GET; 
 
             /* We already read the message body (if any), so don't allow 
              * the redirected request to think it has one. We can ignore 
              * Transfer-Encoding, since we used REQUEST_CHUNKED_ERROR. 
              */ 
-            ap_table_unset(r->headers_in, "Content-Length"); 
+            apr_table_unset(r->headers_in, "Content-Length"); 
 
             ap_internal_redirect_handler(location, r); 
             return OK; 
@@ -1009,40 +1028,313 @@ static int cgid_handler(request_rec *r)
             return HTTP_MOVED_TEMPORARILY; 
         } 
 
-        ap_send_http_header(r); 
         if (!r->header_only) { 
-            ap_send_fb(script, r); 
+            bb = apr_brigade_create(r->pool);
+            b = apr_bucket_pipe_create(tempsock);
+            APR_BRIGADE_INSERT_TAIL(bb, b);
+            b = apr_bucket_eos_create();
+            APR_BRIGADE_INSERT_TAIL(bb, b);
+            ap_pass_brigade(r->output_filters, bb);
         } 
-        ap_bclose(script); 
     } 
 
-    if (script && nph) { 
-        ap_send_fb(script, r); 
+    if (nph) {
+        bb = apr_brigade_create(r->pool);
+        b = apr_bucket_pipe_create(tempsock);
+        APR_BRIGADE_INSERT_TAIL(bb, b);
+        b = apr_bucket_eos_create();
+        APR_BRIGADE_INSERT_TAIL(bb, b);
+        ap_pass_brigade(r->output_filters, bb);
     } 
 
+    apr_file_close(tempsock);
+
     return OK; /* NOT r->status, even if it has changed. */ 
 } 
 
-static const handler_rec cgid_handlers[] = 
-{ 
-    {CGI_MAGIC_TYPE, cgid_handler}, 
-    {"cgi-script", cgid_handler}, 
-    {NULL} 
-};
 
-static void register_hook(void)
+
+
+/*============================================================================
+ *============================================================================
+ * This is the beginning of the cgi filter code moved from mod_include. This
+ *   is the code required to handle the "exec" SSI directive.
+ *============================================================================
+ *============================================================================*/
+static int include_cgi(char *s, request_rec *r, ap_filter_t *next,
+                       apr_bucket *head_ptr, apr_bucket **inserted_head)
+{
+    request_rec *rr = ap_sub_req_lookup_uri(s, r, next);
+    int rr_status;
+    apr_bucket  *tmp_buck, *tmp2_buck;
+
+    if (rr->status != HTTP_OK) {
+        ap_destroy_sub_req(rr);
+        return -1;
+    }
+
+    /* No hardwired path info or query allowed */
+
+    if ((rr->path_info && rr->path_info[0]) || rr->args) {
+        ap_destroy_sub_req(rr);
+        return -1;
+    }
+    if (rr->finfo.filetype != APR_REG) {
+        ap_destroy_sub_req(rr);
+        return -1;
+    }
+
+    /* Script gets parameters of the *document*, for back compatibility */
+
+    rr->path_info = r->path_info;       /* hard to get right; see mod_cgi.c */
+    rr->args = r->args;
+
+    /* Force sub_req to be treated as a CGI request, even if ordinary
+     * typing rules would have called it something else.
+     */
+
+    rr->content_type = CGI_MAGIC_TYPE;
+
+    /* Run it. */
+
+    rr_status = ap_run_sub_req(rr);
+    if (ap_is_HTTP_REDIRECT(rr_status)) {
+        apr_size_t len_loc;
+        const char *location = apr_table_get(rr->headers_out, "Location");
+
+        location = ap_escape_html(rr->pool, location);
+        len_loc = strlen(location);
+
+        /* XXX: if most of this stuff is going to get copied anyway,
+         * it'd be more efficient to pstrcat it into a single pool buffer
+         * and a single pool bucket */
+
+        tmp_buck = apr_bucket_immortal_create("<A HREF=\"", sizeof("<A HREF=\""));
+        APR_BUCKET_INSERT_BEFORE(head_ptr, tmp_buck);
+        tmp2_buck = apr_bucket_heap_create(location, len_loc, 1);
+        APR_BUCKET_INSERT_BEFORE(head_ptr, tmp2_buck);
+        /* XXX: this looks like a bug: should be sizeof - 1 */
+        tmp2_buck = apr_bucket_immortal_create("\">", sizeof("\">"));
+        APR_BUCKET_INSERT_BEFORE(head_ptr, tmp2_buck);
+        tmp2_buck = apr_bucket_heap_create(location, len_loc, 1);
+        APR_BUCKET_INSERT_BEFORE(head_ptr, tmp2_buck);
+        /* XXX: this looks like a bug: should be sizeof - 1 */
+        tmp2_buck = apr_bucket_immortal_create("</A>", sizeof("</A>"));
+        APR_BUCKET_INSERT_BEFORE(head_ptr, tmp2_buck);
+
+        if (*inserted_head == NULL) {
+            *inserted_head = tmp_buck;
+        }
+    }
+
+    ap_destroy_sub_req(rr);
+
+    return 0;
+}
+
+
+/* This is the special environment used for running the "exec cmd="
+ *   variety of SSI directives.
+ */
+static void add_ssi_vars(request_rec *r, ap_filter_t *next)
+{
+    apr_table_t *e = r->subprocess_env;
+
+    if (r->path_info && r->path_info[0] != '\0') {
+        request_rec *pa_req;
+
+        apr_table_setn(e, "PATH_INFO", ap_escape_shell_cmd(r->pool, r->path_info));
+
+        pa_req = ap_sub_req_lookup_uri(ap_escape_uri(r->pool, r->path_info), r, next);
+        if (pa_req->filename) {
+            apr_table_setn(e, "PATH_TRANSLATED",
+                           apr_pstrcat(r->pool, pa_req->filename, pa_req->path_info, NULL));
+        }
+        ap_destroy_sub_req(pa_req);
+    }
+
+    if (r->args) {
+        char *arg_copy = apr_pstrdup(r->pool, r->args);
+
+        apr_table_setn(e, "QUERY_STRING", r->args);
+        ap_unescape_url(arg_copy);
+        apr_table_setn(e, "QUERY_STRING_UNESCAPED", ap_escape_shell_cmd(r->pool, arg_copy));
+    }
+}
+
+static int include_cmd(include_ctx_t *ctx, apr_bucket_brigade **bb, char *command,
+                       request_rec *r, ap_filter_t *f)
 {
-    ap_hook_post_config(cgid_init, NULL, NULL, AP_HOOK_MIDDLE);
+    char **env; 
+    const char *location; 
+    int sd;
+    apr_status_t rc = APR_SUCCESS; 
+    int retval;
+    apr_bucket_brigade *bcgi;
+    apr_bucket *b;
+    struct sockaddr_un unix_addr;
+    apr_file_t *tempsock = NULL;
+    cgid_server_conf *conf = ap_get_module_config(r->server->module_config,
+                                                  &cgid_module); 
+
+    add_ssi_vars(r, f->next);
+    env = ap_create_environment(r->pool, r->subprocess_env);
+
+    if ((sd = socket(AF_UNIX, SOCK_STREAM, 0)) < 0) {
+            return log_scripterror(r, conf, HTTP_INTERNAL_SERVER_ERROR, 0, 
+                                   "unable to create socket to cgi daemon");
+    }
+
+    memset(&unix_addr, 0, sizeof(unix_addr));
+    unix_addr.sun_family = AF_UNIX;
+    strcpy(unix_addr.sun_path, conf->sockname);
+
+    if (connect(sd, (struct sockaddr *)&unix_addr, sizeof(unix_addr)) < 0) {
+            return log_scripterror(r, conf, HTTP_INTERNAL_SERVER_ERROR, 0, 
+                                   "unable to connect to cgi daemon");
+    } 
+
+    SPLIT_AND_PASS_PRETAG_BUCKETS(*bb, ctx, f->next, rc);
+    if (rc != APR_SUCCESS) {
+        return rc;
+    }
+
+    send_req(sd, r, command, env, SSI_REQ); 
+
+    /* We are putting the tempsock variable into a file so that we can use
+     * a pipe bucket to send the data to the client.
+     */
+    apr_os_file_put(&tempsock, &sd, r->pool);
+
+    if ((retval = ap_setup_client_block(r, REQUEST_CHUNKED_ERROR))) 
+        return retval; 
+    
+    location = apr_table_get(r->headers_out, "Location"); 
+
+    if (location && location[0] == '/' && r->status == 200) { 
+        char argsbuffer[HUGE_STRING_LEN]; 
+
+        /* Soak up all the script output */ 
+        while (apr_file_gets(argsbuffer, HUGE_STRING_LEN, 
+                             tempsock) == APR_SUCCESS) { 
+            continue; 
+        } 
+        /* This redirect needs to be a GET no matter what the original 
+         * method was. 
+         */ 
+        r->method = apr_pstrdup(r->pool, "GET"); 
+        r->method_number = M_GET; 
+
+        /* We already read the message body (if any), so don't allow 
+         * the redirected request to think it has one. We can ignore 
+         * Transfer-Encoding, since we used REQUEST_CHUNKED_ERROR. 
+         */ 
+        apr_table_unset(r->headers_in, "Content-Length"); 
+
+        ap_internal_redirect_handler(location, r); 
+        return OK; 
+    } 
+    else if (location && r->status == 200) { 
+        /* XX Note that if a script wants to produce its own Redirect 
+         * body, it now has to explicitly *say* "Status: 302" 
+         */ 
+        return HTTP_MOVED_TEMPORARILY; 
+    } 
+
+    if (!r->header_only) { 
+        bcgi = apr_brigade_create(r->pool);
+        b    = apr_bucket_pipe_create(tempsock);
+        APR_BRIGADE_INSERT_TAIL(bcgi, b);
+        ap_pass_brigade(f->next, bcgi);
+    } 
+
+    return 0;
+}
+
+static int handle_exec(include_ctx_t *ctx, apr_bucket_brigade **bb, request_rec *r,
+                       ap_filter_t *f, apr_bucket *head_ptr, apr_bucket **inserted_head)
+{
+    char *tag     = NULL;
+    char *tag_val = NULL;
+    char *file = r->filename;
+    apr_bucket  *tmp_buck;
+    char parsed_string[MAX_STRING_LEN];
+
+    *inserted_head = NULL;
+    if (ctx->flags & FLAG_PRINTING) {
+        if (ctx->flags & FLAG_NO_EXEC) {
+            ap_log_rerror(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, r,
+                      "exec used but not allowed in %s", r->filename);
+            CREATE_ERROR_BUCKET(ctx, tmp_buck, head_ptr, *inserted_head);
+        }
+        else {
+            while (1) {
+                cgid_pfn_gtv(ctx, &tag, &tag_val, 1);
+                if (tag_val == NULL) {
+                    if (tag == NULL) {
+                        return (0);
+                    }
+                    else {
+                        return 1;
+                    }
+                }
+                if (!strcmp(tag, "cmd")) {
+                    cgid_pfn_ps(r, tag_val, parsed_string, sizeof(parsed_string), 1);
+                    if (include_cmd(ctx, bb, parsed_string, r, f) == -1) {
+                        ap_log_rerror(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, r,
+                                    "execution failure for parameter \"%s\" "
+                                    "to tag exec in file %s", tag, r->filename);
+                        CREATE_ERROR_BUCKET(ctx, tmp_buck, head_ptr, *inserted_head);
+                    }
+                    /* just in case some stooge changed directories */
+                }
+                else if (!strcmp(tag, "cgi")) {
+                    apr_status_t retval = APR_SUCCESS;
+
+                    cgid_pfn_ps(r, tag_val, parsed_string, sizeof(parsed_string), 0);
+                    SPLIT_AND_PASS_PRETAG_BUCKETS(*bb, ctx, f->next, retval);
+                    if (retval != APR_SUCCESS) {
+                        return retval;
+                    }
+
+                    if (include_cgi(parsed_string, r, f->next, head_ptr, inserted_head) == -1) {
+                        ap_log_rerror(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, r,
+                                    "invalid CGI ref \"%s\" in %s", tag_val, file);
+                        CREATE_ERROR_BUCKET(ctx, tmp_buck, head_ptr, *inserted_head);
+                    }
+                }
+                else {
+                    ap_log_rerror(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, r,
+                                "unknown parameter \"%s\" to tag exec in %s", tag, file);
+                    CREATE_ERROR_BUCKET(ctx, tmp_buck, head_ptr, *inserted_head);
+                }
+            }
+        }
+    }
+    return 0;
+}
+/*============================================================================
+ *============================================================================
+ * This is the end of the cgi filter code moved from mod_include.
+ *============================================================================
+ *============================================================================*/
+
+
+static void register_hook(apr_pool_t *p)
+{
+    static const char * const aszPre[] = { "mod_include.c", NULL };
+
+    ap_hook_post_config(cgid_init, aszPre, NULL, APR_HOOK_MIDDLE);
+    ap_hook_handler(cgid_handler, NULL, NULL, APR_HOOK_MIDDLE);
 }
 
-module MODULE_VAR_EXPORT cgid_module = { 
+module AP_MODULE_DECLARE_DATA cgid_module = { 
     STANDARD20_MODULE_STUFF, 
     NULL, /* dir config creater */ 
     NULL, /* dir merger --- default is to override */ 
     create_cgid_config, /* server config */ 
     merge_cgid_config, /* merge server config */ 
     cgid_cmds, /* command table */ 
-    cgid_handlers, /* handlers */ 
     register_hook /* register_handlers */ 
 };