Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT sourceware DOT cygnus DOT com>
List-Archive: <http://sourceware.cygnus.com/ml/cygwin-developers/>
List-Post: <mailto:cygwin-developers AT sourceware DOT cygnus DOT com>
List-Help: <mailto:cygwin-developers-help AT sourceware DOT cygnus DOT com>, <http://sourceware.cygnus.com/ml/#faqs>
Sender: cygwin-developers-owner AT sourceware DOT cygnus DOT com
Delivered-To: mailing list cygwin-developers AT sourceware DOT cygnus DOT com
Date: Mon, 10 Jan 2000 19:28:00 -0500
From: Chris Faylor <cgf AT cygnus DOT com>
To: Corinna Vinschen <corinna AT vinschen DOT de>
Cc: cygdev <cygwin-developers AT sourceware DOT cygnus DOT com>
Subject: Re: access-patch
Message-ID: <20000110192800.A28920@cygnus.com>
Mail-Followup-To: Corinna Vinschen <corinna AT vinschen DOT de>,
	cygdev <cygwin-developers AT sourceware DOT cygnus DOT com>
References: <387A450F DOT 6F2F272A AT vinschen DOT de> <20000110155112 DOT A17460 AT cygnus DOT com> <387A5907 DOT C7FA929E AT vinschen DOT de>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
X-Mailer: Mutt 1.0i
In-Reply-To: <387A5907.C7FA929E@vinschen.de>; from corinna@vinschen.de on Mon, Jan 10, 2000 at 11:11:19PM +0100

On Mon, Jan 10, 2000 at 11:11:19PM +0100, Corinna Vinschen wrote:
>Chris Faylor wrote:
>> On Mon, Jan 10, 2000 at 09:46:07PM +0100, Corinna Vinschen wrote:
>> >+  if (os_being_run == winNT && allow_ntsec)
>> >+    return acl_access (fn, flags);
>> 
>> Is there any reason for the "os_being_run" test?  I don't see it anywhere
>> else in the code.  If this is required we should probably handle it one
>> time early on rather than always checking for it in access().
>
>Asking for os_being_run is a little paranoid, checking for allow_ntsec
>should be enough. You can simply throw it away.

Ok.   I've applied this with that modification.

cgf