Brian Pane wrote:
> If the read of this field is removed, won't we also need to remove
> the code that writes it?
Correct, my mistake... new patch has been attached that also removes
the writing of the status line.
--
ir. Kris Verbeeck
Development Engineer
Ubizen - Ubicenter - Philipssite 5 - 3001 Leuven - Belgium
T: +32 16 28 70 64
F: +32 16 28 70 77
Ubizen - We Secure e-business - www.ubizen.com
--- mod_disk_cache.c Fri Sep 13 09:44:09 2002
+++ mod_disk_cache.c-PATCHED Fri Sep 13 09:52:44 2002
@@ -484,17 +484,6 @@
r->status = atoi(urlbuff); /* Save status line into
request rec */
- rv = apr_file_gets(&urlbuff[0], urllen, dobj->hfd); /* Read status
line */
- if (rv != APR_SUCCESS) {
- /* XXX log message */
- return rv;
- }
-
- if ((temp = strchr(&urlbuff[0], '\n')) != NULL) /* trim off new line
character */
- *temp = '\0'; /* overlay it with the null terminator */
-
- r->status_line = apr_pstrdup(r->pool, urlbuff); /* Save status line
into request rec */
-
h->req_hdrs = apr_table_make(r->pool, 20);
/*
@@ -592,17 +581,6 @@
}
sprintf(statusbuf,"%d", r->status);
buf = apr_pstrcat(r->pool, statusbuf, CRLF, NULL);
- amt = strlen(buf);
- apr_file_write(hfd, buf, &amt);
-
- /* This case only occurs when the content is generated locally */
- if (!r->status_line) {
- r->status_line = ap_get_status_line(r->status);
- }
- buf = apr_pstrcat(r->pool, r->status_line, "\n", NULL);
- amt = strlen(buf);
- apr_file_write(hfd, buf, &amt);
- buf = apr_pstrcat(r->pool, CRLF, NULL);
amt = strlen(buf);
apr_file_write(hfd, buf, &amt);
