* mod_filter: dispatch correctly where dispatch string doesn't exist

      PR 48054
          Trunk patch: n/a (trunk upgraded to use ap_expr for condition testing)
              2.2.x patch: https://issues.apache.org/bugzilla/attachment.cgi?id=24418
                  +1: niq, rpluem, jim

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@892261 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/STATUS b/STATUS
index 19bc96e05ce9ff2b53c5a59ca6c1b3c2b91a98e6..7478f4e33aa80eb3e2183dc97429114ed426333e 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -87,11 +87,6 @@ RELEASE SHOWSTOPPERS:
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
-  * mod_filter: dispatch correctly where dispatch string doesn't exist
-    PR 48054
-    Trunk patch: n/a (trunk upgraded to use ap_expr for condition testing)
-    2.2.x patch: https://issues.apache.org/bugzilla/attachment.cgi?id=24418
-    +1: niq, rpluem, jim
 
 
 PATCHES PROPOSED TO BACKPORT FROM TRUNK:

diff --git a/modules/filters/mod_filter.c b/modules/filters/mod_filter.c
index 3ba13c954ec74556d0d4ed894bd5044d7c600055..7389364246748ff9bd92416f21271d9802c2fee3 100644 (file)
--- a/modules/filters/mod_filter.c
+++ b/modules/filters/mod_filter.c
@@ -201,9 +201,7 @@ static int filter_lookup(ap_filter_t *f, ap_filter_rec_t *filter)
          * Not sure if there's anything better to do with them
          */
         if (!str) {
-            if (provider->match_type == DEFINED && provider->match.string) {
-                match = 0;
-            }
+            match = 0;
         }
         /* we can't check for NULL in provider as that kills integer 0
          * so we have to test each string/regexp case in the switch