Commit 6b132592 authored by William A. Rowe Jr's avatar William A. Rowe Jr
Browse files

After eliminating unusual whitespace in Unsafe mode (e.g. \f \v), we are left 

with the same behavior in both of these cases. Simplify. Noted by rpluem.

Backports: 1773162


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x-merge-http-strict@1773163 13f79535-47bb-0310-9956-ffa450edef68
parent 31d1ab2a

modules/http/http_filters.c

+1 −11
Original line number Diff line number Diff line
@@ -715,17 +715,7 @@ static int check_header(void *arg, const char *name, const char *val) 
        return 0;

    }



    if (ctx->strict) { 

    test = ap_scan_http_field_content(val);

    }

    else {

        /* Simply terminate scanning on a CTL char, allowing whitespace */

        test = val;

        do {

            while (*test == ' ' || *test == '\t') test++;

            test = ap_scan_vchar_obstext(test);

        } while (*test == ' ' || *test == '\t');

    }

    if (*test) {

        ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, ctx->r, APLOGNO(02430)

                      "Response header '%s' value of '%s' contains invalid "