Commit e73f480d authored by Eric Covener's avatar Eric Covener
Browse files

Merge r1775827 from 2.4.x: 

Merge r1775199, r1775664 from trunk:

Fix strict Host: header checking on EBCDIC

on zOS, isascii() really means 7 bit ascii, but our strings
are in ebcdic for 99.95% of the lifetime of the server.



remove initial isascii check entirely

We are already checking an even narrower set of characters
just below.


Submitted by: covener
Reviewed/backported by: jim



git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x-merge-http-strict@1777247 13f79535-47bb-0310-9956-ffa450edef68
parent 7e85ac80

server/vhost.c

+1 −4
Original line number Diff line number Diff line
@@ -757,10 +757,7 @@ static apr_status_t strict_hostname_check(request_rec *r, char *host) 
    int is_dotted_decimal = 1, leading_zeroes = 0, dots = 0;



    for (ch = host; *ch; ch++) {

        if (!apr_isascii(*ch)) {

            goto bad;

        }

        else if (apr_isalpha(*ch) || *ch == '-') {

        if (apr_isalpha(*ch) || *ch == '-') {

            is_dotted_decimal = 0;

        }

        else if (ch[0] == '.') {