]> git.ipfire.org Git - thirdparty/apache/httpd.git/commitdiff
*Sigh.* Again, removing UTIL_URI_REGEX stuff, this time from 2.0.
authorVictor J. Orlikowski <orlikowski@apache.org>
Wed, 4 Apr 2001 07:00:38 +0000 (07:00 +0000)
committerVictor J. Orlikowski <orlikowski@apache.org>
Wed, 4 Apr 2001 07:00:38 +0000 (07:00 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@88715 13f79535-47bb-0310-9956-ffa450edef68

server/util_uri.c

index 2189e9710da78df262aef81166341134b3b31146..2feb23c2b6068f7d4379aa480dad4f91ba9808b6 100644 (file)
@@ -161,184 +161,6 @@ AP_DECLARE(char *) ap_unparse_uri_components(apr_pool_t *p, const uri_components
     return ret;
 }
 
-/* The regex version of parse_uri_components has the advantage that it is
- * relatively easy to understand and extend.  But it has the disadvantage
- * that the regexes are complex enough that regex libraries really
- * don't do a great job with them performancewise.
- *
- * The default is a hand coded scanner that is two orders of magnitude
- * faster.
- */
-#ifdef UTIL_URI_REGEX
-
-static regex_t re_uri;
-static regex_t re_hostpart;
-
-AP_DECLARE(void) ap_util_uri_init(void)
-{
-    int ret;
-    const char *re_str;
-
-    /* This is a modified version of the regex that appeared in
-     * draft-fielding-uri-syntax-01.  It doesnt allow the uri to contain a
-     * scheme but no hostinfo or vice versa. 
-     *
-     * draft-fielding-uri-syntax-01.txt, section 4.4 tells us:
-     *
-     *     Although the BNF defines what is allowed in each component, it is
-     *     ambiguous in terms of differentiating between a site component and
-     *     a path component that begins with two slash characters.
-     *  
-     * RFC2068 disambiguates this for the Request-URI, which may only ever be
-     * the "abs_path" portion of the URI.  So a request "GET //foo/bar
-     * HTTP/1.1" is really referring to the path //foo/bar, not the host foo,
-     * path /bar.  Nowhere in RFC2068 is it possible to have a scheme but no
-     * hostinfo or a hostinfo but no scheme.  (Unless you're proxying a
-     * protocol other than HTTP, but this parsing engine probably won't work
-     * for other protocols.)
-     *
-     *         12            3          4       5   6        7 8 */
-    re_str = "^(([^:/?#]+)://([^/?#]*))?([^?#]*)(\\?([^#]*))?(#(.*))?$";
-    /*          ^scheme--^   ^site---^  ^path--^   ^query^    ^frag */
-    if ((ret = regcomp(&re_uri, re_str, REG_EXTENDED)) != 0) {
-       char line[1024];
-
-       /* Make a readable error message */
-       ret = regerror(ret, &re_uri, line, sizeof line);
-       ap_log_error(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, NULL,
-               "Internal error: regcomp(\"%s\") returned non-zero (%s) - "
-               "possibly due to broken regex lib! "
-               "Did you define WANTHSREGEX=yes?",
-               re_str, line);
-
-       exit(1);
-    }
-
-    /* This is a sub-RE which will break down the hostinfo part,
-     * i.e., user, password, hostname and port.
-     * $          12      3 4        5       6 7    */
-    re_str    = "^(([^:]*)(:(.*))?@)?([^@:]*)(:([0-9]*))?$";
-    /*             ^^user^ :pw        ^host^ ^:[port]^ */
-    if ((ret = regcomp(&re_hostpart, re_str, REG_EXTENDED)) != 0) {
-       char line[1024];
-
-       /* Make a readable error message */
-       ret = regerror(ret, &re_hostpart, line, sizeof line);
-       ap_log_error(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, NULL,
-               "Internal error: regcomp(\"%s\") returned non-zero (%s) - "
-               "possibly due to broken regex lib! "
-               "Did you define WANTHSREGEX=yes?",
-               re_str, line);
-
-       exit(1);
-    }
-}
-
-
-/* parse_uri_components():
- * Parse a given URI, fill in all supplied fields of a uri_components
- * structure. This eliminates the necessity of extracting host, port,
- * path, query info repeatedly in the modules.
- * Side effects:
- *  - fills in fields of uri_components *uptr
- *  - none on any of the r->* fields
- */
-AP_DECLARE(int) ap_parse_uri_components(apr_pool_t *p, const char *uri, uri_components *uptr)
-{
-    int ret;
-    regmatch_t match[10];      /* This must have at least as much elements
-                               * as there are braces in the re_strings */
-
-    ap_assert (uptr != NULL);
-
-    /* Initialize the structure. parse_uri() and parse_uri_components()
-     * can be called more than once per request.
-     */
-    memset (uptr, '\0', sizeof(*uptr));
-    uptr->is_initialized = 1;
-
-    ret = ap_regexec(&re_uri, uri, re_uri.re_nsub + 1, match, 0);
-
-    if (ret != 0) {
-       ap_log_error(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, NULL,
-                    "ap_regexec() could not parse uri (\"%s\")",
-                   uri);
-
-       return HTTP_BAD_REQUEST;
-    }
-
-    if (match[2].rm_so != match[2].rm_eo)
-       uptr->scheme = apr_pstrndup (p, uri+match[2].rm_so, match[2].rm_eo - match[2].rm_so);
-
-    /* empty hostinfo is valid, that's why we test $1 but use $3 */
-    if (match[1].rm_so != match[1].rm_eo)
-       uptr->hostinfo = apr_pstrndup (p, uri+match[3].rm_so, match[3].rm_eo - match[3].rm_so);
-
-    if (match[4].rm_so != match[4].rm_eo)
-       uptr->path = apr_pstrndup (p, uri+match[4].rm_so, match[4].rm_eo - match[4].rm_so);
-
-    /* empty query string is valid, that's why we test $5 but use $6 */
-    if (match[5].rm_so != match[5].rm_eo)
-       uptr->query = apr_pstrndup (p, uri+match[6].rm_so, match[6].rm_eo - match[6].rm_so);
-
-    /* empty fragment is valid, test $7 use $8 */
-    if (match[7].rm_so != match[7].rm_eo)
-       uptr->fragment = apr_pstrndup (p, uri+match[8].rm_so, match[8].rm_eo - match[8].rm_so);
-
-    if (uptr->hostinfo) {
-       /* Parse the hostinfo part to extract user, password, host, and port */
-       ret = ap_regexec(&re_hostpart, uptr->hostinfo, re_hostpart.re_nsub + 1, match, 0);
-       if (ret != 0) {
-           ap_log_error(APLOG_MARK, APLOG_NOERRNO|APLOG_ERR, 0, NULL,
-                    "ap_regexec() could not parse (\"%s\") as host part",
-                   uptr->hostinfo);
-
-           return HTTP_BAD_REQUEST;
-       }
-
-       /* $      12      3 4        5       6 7            */
-       /*      "^(([^:]*)(:(.*))?@)?([^@:]*)(:([0-9]*))?$" */
-       /*         ^^user^ :pw        ^host^ ^:[port]^      */
-
-       /* empty user is valid, that's why we test $1 but use $2 */
-       if (match[1].rm_so != match[1].rm_eo)
-           uptr->user = apr_pstrndup (p, uptr->hostinfo+match[2].rm_so, match[2].rm_eo - match[2].rm_so);
-
-       /* empty password is valid, test $3 but use $4 */
-       if (match[3].rm_so != match[3].rm_eo)
-           uptr->password = apr_pstrndup (p, uptr->hostinfo+match[4].rm_so, match[4].rm_eo - match[4].rm_so);
-
-       /* empty hostname is valid, and implied by the existence of hostinfo */
-       uptr->hostname = apr_pstrndup (p, uptr->hostinfo+match[5].rm_so, match[5].rm_eo - match[5].rm_so);
-
-       if (match[6].rm_so != match[6].rm_eo) {
-           /* Note that the port string can be empty.
-            * If it is, we use the default port associated with the scheme
-            */
-           uptr->port_str = apr_pstrndup (p, uptr->hostinfo+match[7].rm_so, match[7].rm_eo - match[7].rm_so);
-           if (uptr->port_str[0] != '\0') {
-               char *endstr;
-               int port;
-
-               port = strtol(uptr->port_str, &endstr, 10);
-               uptr->port = port;
-               if (*endstr != '\0') {
-                   /* Invalid characters after ':' found */
-                   return HTTP_BAD_REQUEST;
-               }
-           }
-           else {
-               uptr->port = uptr->scheme ? ap_default_port_for_scheme(uptr->scheme) : DEFAULT_HTTP_PORT;
-           }
-       }
-    }
-
-    if (ret == 0)
-       ret = HTTP_OK;
-    return ret;
-}
-#else
-
 /* Here is the hand-optimized parse_uri_components().  There are some wild
  * tricks we could pull in assembly language that we don't pull here... like we
  * can do word-at-time scans for delimiter characters using the same technique
@@ -540,4 +362,3 @@ AP_DECLARE(int) ap_parse_hostinfo_components(apr_pool_t *p, const char *hostinfo
     }
     return HTTP_BAD_REQUEST;
 }
-#endif