httpd-bugs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 9727] New: - Double quotes should be flagged as T_HTTP_TOKEN_STOP
Date Sun, 09 Jun 2002 12:09:53 GMT
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=9727>.
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=9727

Double quotes should be flagged as T_HTTP_TOKEN_STOP

           Summary: Double quotes should be flagged as T_HTTP_TOKEN_STOP
           Product: Apache httpd-2.0
           Version: 2.0.36
          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 
2.0.36: 
 
--- server/gen_test_char.c~	Tue Apr  9 14:12:10 2002 
+++ server/gen_test_char.c	Sun Jun  9 14:58:59 2002 
@@ -130,8 +130,8 @@ 
             flags |= T_OS_ESCAPE_PATH; 
         } 
  
-        /* these are the "tspecials" from RFC2068 */ 
-        if (apr_iscntrl(c) || strchr(" \t()<>@,;:\\/[]?={}", c)) { 
+        /* these are the "tspecials" (RFC2068) or "separators" (RFC2616) */ 
+        if (apr_iscntrl(c) || strchr(" \t()<>@,;:\\\"/[]?={}", c)) { 
             flags |= T_HTTP_TOKEN_STOP; 
         }

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


Mime
View raw message