Changeset 433

Timestamp:

07/09/2005 08:17:40 PM (3 years ago)

Author:

jan

Message:

ah, don't care about the valid chars, control-chars, 127 and 255 are out

Files:

• branches/lighttpd-1.3.x/src/request.c (modified) (1 diff)

branches/lighttpd-1.3.x/src/request.c

@@ -260 +260 @@
 }
 
-int request_uri_is_valid_char(char c) {
-        /* RFC 2396 - Appendix A */
-        
-        /* alphanum */
-        if (light_isalnum(c)) return 1;
-        if (c < 0) return 1; /* no-ascii chars are ok */
-        
-        switch(c) {
-                /* reserved */
-        case ';':
-        case '/':
-        case '?': 
-        case ':':
-        case '@':
-        case '&':
-        case '=':
-        case '+': /* only in Query part it is rewritten to ' ' (space) */
-        case '$':
-        case ',':
-                
-                /* mark */
-        case '-':
-        case '_':
-        case '.':
-        case '!':
-        case '~':
-        case '*':
-        case '\'':
-        case '(':
-        case ')':
-                
-                /* escaped */
-        case '%':
-                
-                /* fragment, should not be out in the wild $*/
-        case '#': 
-                
-                /* non RFC */
-        case '[': 
-        case ']':
-        case '|':       
-                return 1;
-        }
-        
-        return 0;
+int request_uri_is_valid_char(unsigned char c) {
+        if (c <= 32) return 0;
+        if (c == 127) return 0;
+        if (c == 255) return 0;
+        
+        return 1;
 }