Re: svn commit: r1772418 - /httpd/httpd/trunk/modules/http/http_filters.c
probably wiser ways to fix but I didn't want to sit on it.
On Fri, Dec 2, 2016 at 7:10 PM, wrote:
> Author: covener
> Date: Sat Dec 3 00:10:31 2016
> New Revision: 1772418
>
> URL: http://svn.apache.org/viewvc?rev=1772418&view=rev
> Log:
> loop in checking response headers
>
> w/ HTTPProtocolOptions Unsafe
>
> Modified:
> httpd/httpd/trunk/modules/http/http_filters.c
>
> Modified: httpd/httpd/trunk/modules/http/http_filters.c
> URL:
> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http/http_filters.c?rev=1772418&r1=1772417&r2=1772418&view=diff
> ==
> --- httpd/httpd/trunk/modules/http/http_filters.c (original)
> +++ httpd/httpd/trunk/modules/http/http_filters.c Sat Dec 3 00:10:31 2016
> @@ -667,6 +667,7 @@ static int check_header(void *arg, const
> /* 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');
> }
>
>
--
Eric Covener
cove...@gmail.com
Re: svn commit: r1772418 - /httpd/httpd/trunk/modules/http/http_filters.c
That is code I'm less familiar with, but will give it careful scrutiny
tomorrow.
On Fri, Dec 2, 2016 at 6:12 PM, Eric Covener wrote:
> probably wiser ways to fix but I didn't want to sit on it.
>
> On Fri, Dec 2, 2016 at 7:10 PM, wrote:
> > Author: covener
> > Date: Sat Dec 3 00:10:31 2016
> > New Revision: 1772418
> >
> > URL: http://svn.apache.org/viewvc?rev=1772418&view=rev
> > Log:
> > loop in checking response headers
> >
> > w/ HTTPProtocolOptions Unsafe
> >
> > Modified:
> > httpd/httpd/trunk/modules/http/http_filters.c
> >
> > Modified: httpd/httpd/trunk/modules/http/http_filters.c
> > URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/
> http/http_filters.c?rev=1772418&r1=1772417&r2=1772418&view=diff
> >
> ==
> > --- httpd/httpd/trunk/modules/http/http_filters.c (original)
> > +++ httpd/httpd/trunk/modules/http/http_filters.c Sat Dec 3 00:10:31
> 2016
> > @@ -667,6 +667,7 @@ static int check_header(void *arg, const
> > /* 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');
> > }
> >
> >
>
>
>
> --
> Eric Covener
> cove...@gmail.com
>
Re: svn commit: r1772418 - /httpd/httpd/trunk/modules/http/http_filters.c
FWIW, did you actually fix this on truck and convey the backport
svn rev no?
On Fri, Dec 2, 2016 at 6:25 PM, William A Rowe Jr
wrote:
> That is code I'm less familiar with, but will give it careful scrutiny
> tomorrow.
>
>
> On Fri, Dec 2, 2016 at 6:12 PM, Eric Covener wrote:
>
>> probably wiser ways to fix but I didn't want to sit on it.
>>
>> On Fri, Dec 2, 2016 at 7:10 PM, wrote:
>> > Author: covener
>> > Date: Sat Dec 3 00:10:31 2016
>> > New Revision: 1772418
>> >
>> > URL: http://svn.apache.org/viewvc?rev=1772418&view=rev
>> > Log:
>> > loop in checking response headers
>> >
>> > w/ HTTPProtocolOptions Unsafe
>> >
>> > Modified:
>> > httpd/httpd/trunk/modules/http/http_filters.c
>> >
>> > Modified: httpd/httpd/trunk/modules/http/http_filters.c
>> > URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http/
>> http_filters.c?rev=1772418&r1=1772417&r2=1772418&view=diff
>> >
>> ==
>> > --- httpd/httpd/trunk/modules/http/http_filters.c (original)
>> > +++ httpd/httpd/trunk/modules/http/http_filters.c Sat Dec 3 00:10:31
>> 2016
>> > @@ -667,6 +667,7 @@ static int check_header(void *arg, const
>> > /* 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');
>> > }
>> >
>> >
>>
>>
>>
>> --
>> Eric Covener
>> cove...@gmail.com
>>
>
>
Re: svn commit: r1772418 - /httpd/httpd/trunk/modules/http/http_filters.c
On Fri, Dec 2, 2016 at 6:26 PM, William A Rowe Jr
wrote:
> FWIW, did you actually fix this on truck and convey the backport
> svn rev no?
>
I see you sort-of have... please convey the svn commit r1772418 to your
branch commit --revprop svn:log history, and I think we are all good.
> On Fri, Dec 2, 2016 at 6:25 PM, William A Rowe Jr
> wrote:
>
>> That is code I'm less familiar with, but will give it careful scrutiny
>> tomorrow.
>>
>>
>> On Fri, Dec 2, 2016 at 6:12 PM, Eric Covener wrote:
>>
>>> probably wiser ways to fix but I didn't want to sit on it.
>>>
>>> On Fri, Dec 2, 2016 at 7:10 PM, wrote:
>>> > Author: covener
>>> > Date: Sat Dec 3 00:10:31 2016
>>> > New Revision: 1772418
>>> >
>>> > URL: http://svn.apache.org/viewvc?rev=1772418&view=rev
>>> > Log:
>>> > loop in checking response headers
>>> >
>>> > w/ HTTPProtocolOptions Unsafe
>>> >
>>> > Modified:
>>> > httpd/httpd/trunk/modules/http/http_filters.c
>>> >
>>> > Modified: httpd/httpd/trunk/modules/http/http_filters.c
>>> > URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http/
>>> http_filters.c?rev=1772418&r1=1772417&r2=1772418&view=diff
>>> >
>>> ==
>>> > --- httpd/httpd/trunk/modules/http/http_filters.c (original)
>>> > +++ httpd/httpd/trunk/modules/http/http_filters.c Sat Dec 3 00:10:31
>>> 2016
>>> > @@ -667,6 +667,7 @@ static int check_header(void *arg, const
>>> > /* 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');
>>> > }
>>> >
>>> >
>>>
>>>
>>>
>>> --
>>> Eric Covener
>>> cove...@gmail.com
>>>
>>
>>
>
Re: svn commit: r1772418 - /httpd/httpd/trunk/modules/http/http_filters.c
On Fri, Dec 2, 2016 at 7:28 PM, William A Rowe Jr wrote: > On Fri, Dec 2, 2016 at 6:26 PM, William A Rowe Jr > wrote: >> >> FWIW, did you actually fix this on truck and convey the backport >> svn rev no? > > > I see you sort-of have... please convey the svn commit r1772418 to your > branch commit --revprop svn:log history, and I think we are all good. It's already in the log unless I'm misunderstanding.
Re: svn commit: r1772418 - /httpd/httpd/trunk/modules/http/http_filters.c
You are correct, I read the commits out of sequence. Thx! On Dec 2, 2016 18:40, "Eric Covener" wrote: > On Fri, Dec 2, 2016 at 7:28 PM, William A Rowe Jr > wrote: > > On Fri, Dec 2, 2016 at 6:26 PM, William A Rowe Jr > > wrote: > >> > >> FWIW, did you actually fix this on truck and convey the backport > >> svn rev no? > > > > > > I see you sort-of have... please convey the svn commit r1772418 to your > > branch commit --revprop svn:log history, and I think we are all good. > > It's already in the log unless I'm misunderstanding. >
