Return-Path: 
 <dev-return-35381-apmail-httpd-dev-archive=httpd.apache.org@httpd.apache.org>
Delivered-To: apmail-httpd-dev-archive@httpd.apache.org
Received: (qmail 42799 invoked by uid 500); 28 Dec 2002 17:20:21 -0000
Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
list-help: <mailto:dev-help@httpd.apache.org>
list-unsubscribe: <mailto:dev-unsubscribe@httpd.apache.org>
list-post: <mailto:dev@httpd.apache.org>
Delivered-To: mailing list dev@httpd.apache.org
Received: (qmail 42786 invoked from network); 28 Dec 2002 17:20:20 -0000
From: =?ISO-8859-1?Q?Andr=E9?= Malo <nd@perlig.de>
Subject: Re: bug 12395
MIME-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: quoted-printable
Organization: TIMTOWTDI
References: <20021004165933.X98886-100000@foem.leiden.webweaving.org>
 <10cd01c2ae83$ba787340$2200a8c0@gce>
Date: Sat, 28 Dec 2002 18:14:27 +0100
To: dev@httpd.apache.org
Message-ID: <n2m-g.1hcy08qwy703$ndparker@news.perlig.de>
User-Agent: Yes!
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N

* Dr. Georg Czedik-Eysenberg wrote:

> But on the other hand it is now about 4 months,
> since I entered the "bug report 12395" into Bugzilla
> (http://nagoya.apache.org/bugzilla/show_bug.cgi?id=3D12395)
> and it is still in the state "NEW".

<snip>

this bug should be easy to fix:

mod_rewrite.c, HEAD, line 1412:

    /*
     *  only do something under runtime if the engine is really enabled,
     *  for this directory, else return immediately!
     */
    if (!(ap_allow_options(r) & (OPT_SYM_LINKS | OPT_SYM_OWNER))) {
        /* FollowSymLinks is mandatory! */
        ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
                     "Options FollowSymLinks or SymLinksIfOwnerMatch is off=
=20
"
                     "which implies that RewriteRule directive is=20
forbidden: "
                     "%s", r->filename);
        return HTTP_FORBIDDEN;
    }
    else {
        /* FollowSymLinks is given, but the user can
         * still turn off the rewriting engine
         */
        if (dconf->state =3D=3D ENGINE_DISABLED) {
            return DECLINED;
        }
    }

the two conditional branches simply have to be swapped. Or better we could=
=20
remove the else-branch and put the disabled-check before the 'if'. Or...?

nd
--=20
my @japh =3D (sub{q~Just~},sub{q~Another~},sub{q~Perl~},sub{q~Hacker~});
my $japh =3D q[sub japh { }]; print join       #########################
 [ $japh =3D~ /{(.)}/] -> [0] =3D> map $_ -> ()  #            Andr=E9 Malo #
=3D> @japh;                                    # http://www.perlig.de/ #