https://bz.apache.org/bugzilla/show_bug.cgi?id=69745
--- Comment #3 from Eric Covener <cove...@gmail.com> ---
(In reply to Eric Covener from comment #2)
> @@ -4258,7 +4297,7 @@ test_str_l:
> break;
> }
>
> - if (p->flags & CONDFLAG_NOTMATCH) {
> + if (p->flags & CONDFLAG_NOTMATCH && rc <= COND_RC_MATCH) {
> rc = !rc;
> }
>
> previously it flipped the return code because the values were only 0 and 1.
> We're now avoiding the flip for the new COND_RC_STATUS_SET(3), which is
> bubbled up as an error no matter what the condition was looking for.
>
> Likely somewhere in the same neighborhood though, thinking here:
>
> @@ -4237,8 +4273,11 @@ test_str_l:
> rewritelog(r, 1, ctx->perdir,
> "RewriteCond: expr='%s' evaluation failed: %s",
> p->pattern - p->pskip, err);
> - rc = 0;
> + rc = COND_RC_NOMATCH;
> }
> + else {
> + rc = COND_RC_MATCH;
> + }
looks like losing the result of the expression here
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscr...@httpd.apache.org
For additional commands, e-mail: bugs-h...@httpd.apache.org
