httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <>
Subject Re: [PATCH] mod_mime.c
Date Thu, 14 Mar 2002 18:01:30 GMT
"RCHAPACH Rochester" <> writes:

> In theme with the "broken module" being allowed in ap_directory_walk()
> and ap_file_walk(), find_ct() of mod_mime.c needs the following change
> to allow a missing filename to pass through to the handlers. I am pretty
> sure that DECLINED is the correct return, but I don't think it matters.

Conceptually I'm having trouble with the concept of "broken module"
here and "older modules" in the first showstopper listed in STATUS.  I
don't know if we mean "module is violating the API and should be
nailed" or "module is using the API in a valid way which is different
than most modules" or something in-between.

If the module is violating the API, we shouldn't try to accomodate it
at all.  Maybe we can figure out a way that type checkers wouldn't
have to validate r->filename.

If for all we know the module is operating properly, then we are
essentially forcing all type checkers to validate r->filename, and the
comment below about "broken modules" is invalid, and we need to
somehow document that a module can't depend on r->filename.

> Index: mod_mime.c
> ===================================================================
> RCS file: /home/cvs/httpd-2.0/modules/http/mod_mime.c
> retrieving revision 1.76
> --- mod_mime.c                    Sat Dec 8 02:02:51 2001
> +++ mod_mime.c    Mon Feb  4 09:30:46 2002
> @@ -736,6 +736,14 @@
>      const char *fn, *type, *charset = NULL;
>      int found_metadata = 0;
> +    /* To allow broken modules to proceed, we allow missing filenames to
> pass.

In need of a clue,
Jeff Trawick |
Born in Roswell... married an alien...

View raw message