Eric Covener wrote:
> On Fri, Jan 6, 2012 at 9:46 AM, Ruediger Pluem <[email protected]> wrote:
>>
>>
>> Eric Covener wrote:
>>> Looks like SuSE backported the trunk fix which misfires on the
>>> internally dummy conn.
>>>
>>> STATUS says we were thinking of flipping this to declined. Looks like
>>> a good idea?
>>
>> Could you please provide some more pointers?
>
>
> I assume SuSe backported our trunk solution inside of mod_rewrite and
> mod_proxy, which is proposed for 2.2.x
But the trunk code looks like
Index: mod_rewrite.c
===================================================================
--- mod_rewrite.c (revision 1209431)
+++ mod_rewrite.c (revision 1209432)
@@ -4419,6 +4419,18 @@
return DECLINED;
}
+ if (strcmp(r->unparsed_uri, "*") == 0) {
+ /* Don't apply rewrite rules to "*". */
+ return DECLINED;
+ }
+
+ /* Check that the URI is valid. */
+ if (!r->uri || r->uri[0] != '/') {
+ ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
+ "Invalid URI in request %s", r->the_request);
+ return HTTP_BAD_REQUEST;
+ }
+
/*
* add the SCRIPT_URL variable to the env. this is a bit complicated
* due to the fact that apache uses subrequests and internal redirects
So IMHO this error should not happen as I assume that the value of
r->unparsed_uri should be "*" in this case.
Regards
Rüdiger
