>Number: 1667
>Category: mod_autoindex
>Synopsis: <PRE> emitted before preamble for non-html header file
>Confidential: no
>Severity: non-critical
>Priority: medium
>Responsible: apache
>State: open
>Class: sw-bug
>Submitter-Id: apache
>Arrival-Date: Wed Jan 14 01:40:00 PST 1998
>Last-Modified:
>Originator: [EMAIL PROTECTED]
>Organization:
apache
>Release: 1.3b3
>Environment:
N/A
>Description:
If the file specified by HeaderName is found as plain text (not html)
then the resulting page has a <PRE> in front of the preamble, instead
of after the preamble.
>How-To-Repeat:
The problem is obvious in the source...
>Fix:
Diff -c output for simple patch that fixes it.
It moves the <PRE> output after the preamble output.
*** mod_autoindex.c.orig Wed Nov 12 16:37:45 1997
--- mod_autoindex.c Wed Jan 14 04:18:14 1998
***************
*** 538,544 ****
plaintext = 1;
if (hrule)
rputs("<HR>\n", r);
- rputs("<PRE>\n", r);
}
else if (hrule)
rputs("<HR>\n", r);
--- 538,543 ----
***************
*** 564,569 ****
--- 563,569 ----
else {
char buf[IOBUFSIZE + 1];
int i, n, c, ch;
+ rputs("<PRE>\n", r);
while (!feof(f)) {
do
n = fread(buf, sizeof(char), IOBUFSIZE, f);
%0
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, ]
[you need to include <[EMAIL PROTECTED]> in the Cc line ]
[and leave the subject line UNCHANGED. This is not done]
[automatically because of the potential for mail loops. ]
