Return-Path: Delivered-To: new-httpd-archive@hyperreal.org Received: (qmail 6605 invoked by uid 6000); 26 Aug 1999 15:27:40 -0000 Received: (qmail 6593 invoked from network); 26 Aug 1999 15:27:39 -0000 Received: from gremlin.ics.uci.edu (mmdf@128.195.1.70) by taz.hyperreal.org with SMTP; 26 Aug 1999 15:27:39 -0000 Received: from kiwi.ics.uci.edu ( fielding@kiwi.ics.uci.edu [128.195.21.109] ) by gremlin-relay.ics.uci.edu id aa08363 for ; 26 Aug 1999 08:26 PDT To: new-httpd@apache.org Subject: reverse 2.0: merges gone bad Date: Thu, 26 Aug 1999 08:26:14 -0700 From: "Roy T. Fielding" Message-ID: <199908260826.aa08363@gremlin-relay.ics.uci.edu> Sender: new-httpd-owner@apache.org Precedence: bulk Reply-To: new-httpd@apache.org Reminder to self ... reverse this when done mucking with cvs. ------- Forwarded Message Modified: src/main alloc.c http_config.c http_protocol.c Log: FIXME: These changes are just errors from bad merges in the past. Revision Changes Path 1.4 +7 -7 apache-2.0/src/main/alloc.c Index: alloc.c =================================================================== RCS file: /home/fielding/moot/apache-2.0/src/main/alloc.c,v retrieving revision 1.3 retrieving revision 1.4 diff -u -r1.3 -r1.4 --- alloc.c 1999/08/26 13:31:34 1.3 +++ alloc.c 1999/08/26 15:17:06 1.4 @@ -2074,14 +2074,14 @@ if (pipe_in) { close(in_fds[0]); close(in_fds[1]); - } + } if (pipe_out) { close(out_fds[0]); close(out_fds[1]); - } + } errno = save_errno; return 0; - } + } #ifdef WIN32 @@ -2230,10 +2230,10 @@ close(out_fds[0]); close(out_fds[1]); } - if (pipe_err) { + if (pipe_err) { close(err_fds[0]); close(err_fds[1]); - } + } errno = save_errno; return 0; } @@ -2254,11 +2254,11 @@ close(in_fds[0]); } - if (pipe_err) { + if (pipe_err) { close(err_fds[0]); dup2(err_fds[1], STDERR_FILENO); close(err_fds[1]); - } + } /* HP-UX SIGCHLD fix goes here, if someone will remind me what it is... */ signal(SIGCHLD, SIG_DFL); /* Was that it? */ 1.4 +21 -22 apache-2.0/src/main/http_config.c Index: http_config.c =================================================================== RCS file: /home/fielding/moot/apache-2.0/src/main/http_config.c,v retrieving revision 1.3 retrieving revision 1.4 diff -u -r1.3 -r1.4 --- http_config.c 1999/08/26 14:18:34 1.3 +++ http_config.c 1999/08/26 15:17:07 1.4 @@ -1088,28 +1088,27 @@ /* loop through the access names and find the first one */ - while (access_name[0]) { - filename = ap_make_full_path(r->pool, d, - ap_getword_conf(r->pool, &access_name)); - - if ((f = ap_pcfg_openfile(r->pool, filename)) != NULL) { - - dc = ap_create_per_dir_config(r->pool); - - parms.config_file = f; - - errmsg = ap_srm_command_loop(&parms, dc); - - ap_cfg_closefile(f); - - if (errmsg) { - ap_log_rerror(APLOG_MARK, APLOG_ALERT|APLOG_NOERRNO, r, - "%s: %s", filename, errmsg); - return HTTP_INTERNAL_SERVER_ERROR; - } - *result = dc; - break; - } + while (!f && access_name[0]) { + char * w = ap_getword_conf(r->pool, &access_name); + filename = ap_make_full_path(r->pool, d, w); + f = ap_pcfg_openfile(r->pool, filename); + } + if (f) { + dc = ap_create_per_dir_config(r->pool); + + parms.config_file = f; + + errmsg = ap_srm_command_loop(&parms, dc); + + ap_cfg_closefile(f); + + if (errmsg) { + ap_log_rerror(APLOG_MARK, APLOG_ALERT|APLOG_NOERRNO, r, + "%s: %s", filename, errmsg); + return HTTP_INTERNAL_SERVER_ERROR; + } + *result = dc; + } else if (errno != ENOENT && errno != ENOTDIR) { ap_log_rerror(APLOG_MARK, APLOG_CRIT, r, "%s pcfg_openfile: unable to check htaccess file, " 1.6 +3 -3 apache-2.0/src/main/http_protocol.c Index: http_protocol.c =================================================================== RCS file: /home/fielding/moot/apache-2.0/src/main/http_protocol.c,v retrieving revision 1.5 retrieving revision 1.6 diff -u -r1.5 -r1.6 --- http_protocol.c 1999/08/26 14:53:22 1.5 +++ http_protocol.c 1999/08/26 15:17:10 1.6 @@ -1545,6 +1545,9 @@ r->sent_bodyct = 1; return; } + else { + fixup_vary(r); + } /* * Now that we are ready to send a response, we need to combine the two @@ -1564,9 +1567,6 @@ ap_table_unset(r->headers_out, "Vary"); r->proto_num = HTTP_VERSION(1,0); ap_table_set(r->subprocess_env, "force-response-1.0", "1"); - } - else { - fixup_vary(r); } ap_basic_http_header(r); ------- End of Forwarded Message