On Tue, Dec 13, 2016 at 12:20 AM, Yann Ylavic <[email protected]> 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;
> _