You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Jim Jagielski <ji...@jaguNET.com> on 1998/05/03 02:57:08 UTC

[PATCH] PR#2112

Possible patch... I felt this was safer than using ap_str_tolower
on apply_to

Index: src/modules/standard/mod_autoindex.c
===================================================================
RCS file: /export/home/cvs/apache-1.3/src/modules/standard/mod_autoindex.c,v
retrieving revision 1.75
diff -u -r1.75 mod_autoindex.c
--- mod_autoindex.c	1998/04/15 17:09:29	1.75
+++ mod_autoindex.c	1998/05/03 00:55:43
@@ -437,13 +437,13 @@
 	    else if (!path_only) {
 		if (!content_encoding) {
 		    if (p->type == BY_TYPE) {
-			if (content_type && !ap_strcmp_match(content_type, p->apply_to))
+			if (content_type && !ap_strcasecmp_match(content_type, p->apply_to))
 			    return p->data;
 		    }
 		}
 		else {
 		    if (p->type == BY_ENCODING) {
-			if (!ap_strcmp_match(content_encoding, p->apply_to))
+			if (!ap_strcasecmp_match(content_encoding, p->apply_to))
 			    return p->data;
 		    }
 		}
-- 
===========================================================================
   Jim Jagielski   |||   jim@jaguNET.com   |||   http://www.jaguNET.com/
            "That's no ordinary rabbit... that's the most foul,
            cruel and bad-tempered rodent you ever laid eyes on"

Re: [PATCH] PR#2112

Posted by Dean Gaudet <dg...@arctic.org>.
+1... no sense trying to make mod_autoindex any faster, it just needs a
rewrite. 

Dean

On Sat, 2 May 1998, Jim Jagielski wrote:

> Possible patch... I felt this was safer than using ap_str_tolower
> on apply_to
> 
> Index: src/modules/standard/mod_autoindex.c
> ===================================================================
> RCS file: /export/home/cvs/apache-1.3/src/modules/standard/mod_autoindex.c,v
> retrieving revision 1.75
> diff -u -r1.75 mod_autoindex.c
> --- mod_autoindex.c	1998/04/15 17:09:29	1.75
> +++ mod_autoindex.c	1998/05/03 00:55:43
> @@ -437,13 +437,13 @@
>  	    else if (!path_only) {
>  		if (!content_encoding) {
>  		    if (p->type == BY_TYPE) {
> -			if (content_type && !ap_strcmp_match(content_type, p->apply_to))
> +			if (content_type && !ap_strcasecmp_match(content_type, p->apply_to))
>  			    return p->data;
>  		    }
>  		}
>  		else {
>  		    if (p->type == BY_ENCODING) {
> -			if (!ap_strcmp_match(content_encoding, p->apply_to))
> +			if (!ap_strcasecmp_match(content_encoding, p->apply_to))
>  			    return p->data;
>  		    }
>  		}
> -- 
> ===========================================================================
>    Jim Jagielski   |||   jim@jaguNET.com   |||   http://www.jaguNET.com/
>             "That's no ordinary rabbit... that's the most foul,
>             cruel and bad-tempered rodent you ever laid eyes on"
>