https://bz.apache.org/bugzilla/show_bug.cgi?id=69745

--- Comment #2 from Eric Covener <[email protected]> ---
@@ -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;
+            }

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to