Commit 0a5cd9bc authored by Eric Covener's avatar Eric Covener
Browse files

mod_substitute: use local/native LF for splitting 

On platforms where the APR_ASCII_LF != '\n', like EBCDIC systems,
strmatch or pcre patterns from the source or config will be in
the native encoding, and this module will really only work on 
content in the native encoding.

(mod_substitute runs before mod_charset_lite for a similar reason)

I thought #if APR_CHARSET_EBCDIC or even #ifdef __MVS__ was overkill
here. 




git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1791400 13f79535-47bb-0310-9956-ffa450edef68
parent e12ebd64

CHANGES

+3 −0
Original line number Diff line number Diff line 
                                                         -*- coding: utf-8 -*-

Changes with Apache 2.5.0



  *) mod_substitute: Fix spurious AH01328 (Line too long) errors on EBCDIC

     systems.  [Eric Covener]



  *) mod_http2: client streams that lack the EOF flag get now forcefully

     closed with a RST_STREAM (NO_ERROR) when the request has been answered.

     [Stefan Eissing]

modules/filters/mod_substitute.c

+1 −1
Original line number Diff line number Diff line
@@ -453,7 +453,7 @@ static apr_status_t substitute_filter(ap_filter_t *f, apr_bucket_brigade *bb) 
            else {

                int num = 0;

                while (bytes > 0) {

                    nl = memchr(buff, APR_ASCII_LF, bytes);

                    nl = memchr(buff, '\n', bytes);

                    if (nl) {

                        len = (apr_size_t) (nl - buff) + 1;

                        /* split *after* the newline */