On Fri, Jul 29, 2016 at 1:49 PM, Ruediger Pluem <[email protected]> wrote:

>
> > +                else /* Using strict RFC7230 parsing */
> > +                {
> > +                    /* Ensure valid token chars before ':' per RFC 7230
> 3.2.4 */
> > +                    if (!(value = (char
> *)ap_scan_http_token(last_field))
>
> Hm, how could ap_scan_http_token ever return NULL?
>

You are correct, I'd started with ap_get_http_token, but there was no
reason for an extra copy. Thanks for both catches!

Reply via email to