You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bugs@httpd.apache.org by bu...@apache.org on 2002/06/09 14:07:34 UTC
DO NOT REPLY [Bug 9726] New: -
Double quotes should be flagged as T_HTTP_TOKEN_STOP
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=9726>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND
INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=9726
Double quotes should be flagged as T_HTTP_TOKEN_STOP
Summary: Double quotes should be flagged as T_HTTP_TOKEN_STOP
Product: Apache httpd-1.3
Version: 1.3.24
Platform: All
OS/Version: Other
Status: NEW
Severity: Minor
Priority: Other
Component: core
AssignedTo: bugs@httpd.apache.org
ReportedBy: ville.skytta@iki.fi
RFC's 2068 and 2616 list the double quote in the tspecials/separators but they
are not flagged such in gen_test_char.c. Here's the huge patch against
1.3.24:
--- src/main/gen_test_char.c~ Thu Mar 21 18:02:03 2002
+++ src/main/gen_test_char.c Sun Jun 9 14:52:46 2002
@@ -63,8 +63,8 @@
flags |= T_OS_ESCAPE_PATH;
}
- /* these are the "tspecials" from RFC2068 */
- if (ap_iscntrl(c) || strchr(" \t()<>@,;:\\/[]?={}", c)) {
+ /* these are the "tspecials" (RFC2068) or "separators" (RFC2616) */
+ if (ap_iscntrl(c) || strchr(" \t()<>@,;:\\\"/[]?={}", c)) {
flags |= T_HTTP_TOKEN_STOP;
}
printf("%u%c", flags, (c < 255) ? ',' : ' ');
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org