You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Aram Mirzadeh <aw...@qosina.com> on 1995/12/04 04:40:43 UTC

WWW Form Bug Report: "mod_imap.c is broken for gopher:// links" on Linux (fwd)

No ack sent... I don't think we ever had this format working?  Did we?

<Aram>

Forwarded message:
> From nobody@hyperreal.com Sun Dec  3 04:48:36 1995
> Message-Id: <19...@taz.hyperreal.com>
> From: jlewis@inorganic5.fdt.net
> To: awm@qosina.com
> Date: Sun Dec  3  0:49:47 1995
> Subject: WWW Form Bug Report: "mod_imap.c is broken for gopher:// links" on Linux
> 
> Submitter: jlewis@inorganic5.fdt.net
> Operating system: Linux, version: 1.2.13
> Extra Modules used: just the defaults
> URL exhibiting problem: Sorry...I already patched my source.
> 
> Symptoms:
> --
> I have a line in a .map file such as:
> rect gopher://inorganic5.chem.ufl.edu 241,2 404,31
> 
> This works with the old imagemap script from NCSA,
> but with mod_imap in Apache 1.0.0, gopher is not 
> a recognized "type" and its assumed this is a partial
> http:// url.
> 
> This is easily fixed with the following patch:
> 
> diff -ur apache_1.0.0/src/mod_imap.c apache_1.0.0j/src/mod_imap.c
> --- apache_1.0.0/src/mod_imap.c Fri Nov 17 16:43:54 1995
> +++ apache_1.0.0j/src/mod_imap.c        Sun Dec  3 03:29:12 1995
> @@ -213,7 +213,8 @@
>         !strncmp (mapurl, "mailto:", 7) ||
>         !strncmp (mapurl, "ftp:", 4) ||
>         !strncmp (mapurl, "telnet:", 7) ||
> -       !strncmp (mapurl, "news:", 5)) {
> +       !strncmp (mapurl, "news:", 5) ||
> +       !strncmp (mapurl, "gopher:", 7)) {
> 
>         strcpy (redirect, mapurl);
>      }
> 
> 
> But I have to wonder if maybe NCSA did it better by
> just searching for ":" in the URL, or perhaps 
> we should search for "://" || "mailto:", but this 
> is likely to fall into the same trap again down 
> the road.
> 
> Maybe you just assumed gopher was dead :-)
> 
> P.S.  I wrote a patch that I've used since somwhere
> before 0.8.8 that lets you allow "exec cgi" but not
> "exec cmd"...It still applies cleanly to 1.0.0...and 
> it would be really neat to see it included in the 
> Apache source.
> --
> 
> Backtrace:
> --
> 
> --
> 


-- 

			       Aram Mirzadeh
		     awm@qosina.com,awm@hyperreal.com
			http://www.qosina.com/~awm/

	43rd Law of Computing:
        	Anything that can go wr
	Segmentation violation -- Core dumped