Received: (qmail 2235 invoked by uid 2012); 3 Feb 1999 17:22:54 -0000
Message-Id: <19990203172254.2234.qmail@hyperreal.org>
Date: 3 Feb 1999 17:22:54 -0000
From: Brian Smith <opensource_99@hotmail.com>
Reply-To: opensource_99@hotmail.com
To: apbugs@hyperreal.org
Subject: check_dir_access(), reads a->order[31]  (M_INVALID)  where order is defined as 15 ints.
X-Send-Pr-Version: 3.2

>Number:         3821
>Category:       mod_access
>Synopsis:       check_dir_access(), reads a->order[31]  (M_INVALID)  where order is defined as 15 ints.
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    apache
>State:          closed
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Wed Feb  3 09:30:01 PST 1999
>Last-Modified:  Sat Mar 20 16:33:04 PST 1999
>Originator:     opensource_99@hotmail.com
>Organization:
>Release:        1.3.4
>Environment:
apache 1.3.4 win32 download - working in MS VisualC++/Devstudio
>Description:
This hasn't caused me any problems, but I just noticed something I think looks
fishy.

order array is defined to be METHODS long (15), yet when we come into 
check_dir_access for an M_INVALID (31) request, we're reading memory we're 
not supposed to.

This problem might give access voilation errors if not for our memory pools, 
and such. 
I doubt it's a security problem, but access permissions seem to be "random"
for M_INVALID.
>How-To-Repeat:
it's in the code.
>Fix:
check for M_INVALID in check_dir_access.
>Audit-Trail:
State-Changed-From-To: open-closed
State-Changed-By: lars
State-Changed-When: Sat Mar 20 16:33:03 PST 1999
State-Changed-Why:

Fixed. Thanks.

Release-Changed-From-To: 1.3.4 (win32)-1.3.4
Release-Changed-By: lars
Release-Changed-When: Sat Mar 20 16:33:03 PST 1999
>Unformatted:
[In order for any reply to be added to the PR database, ]
[you need to include <apbugs@Apache.Org> in the Cc line ]
[and leave the subject line UNCHANGED.  This is not done]
[automatically because of the potential for mail loops. ]
[If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request ]
[from a developer.                                      ]
[Reply only with text; DO NOT SEND ATTACHMENTS!         ]