On Tue, Dec 13, 2016 at 12:20 AM, Yann Ylavic <ylavic....@gmail.com> wrote:
>
> So I guess r1773861 is OK already, though the below may be simpler/cleaner 
> now:
>
> Index: modules/http/http_filters.c
> ===================================================================
> --- modules/http/http_filters.c    (revision 1773865)
> +++ modules/http/http_filters.c    (working copy)
> @@ -689,13 +689,10 @@ static APR_INLINE int check_headers(request_rec *r
>
>  static int check_headers_recursion(request_rec *r)
>  {
> -    request_rec *rr;
> -    for (rr = r; rr; rr = rr->prev) {
> -        void *dying = NULL;
> -        apr_pool_userdata_get(&dying, "check_headers_recursion", rr->pool);
> -        if (dying) {
> -            return 1;
> -        }
> +    void *dying = NULL;
> +    apr_pool_userdata_get(&dying, "check_headers_recursion", rr->pool);

s/rr/r/ here of course.

> +    if (dying) {
> +        return 1;
>      }
>      apr_pool_userdata_setn("true", "check_headers_recursion", NULL, r->pool);
>      return 0;
> _

Reply via email to