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
Message-ID: <387A5907.C7FA929E@vinschen.de>
Date: Mon, 10 Jan 2000 23:11:19 +0100
From: Corinna Vinschen <corinna AT vinschen DOT de>
X-Mailer: Mozilla 4.7 [en] (WinNT; I)
X-Accept-Language: de,en
MIME-Version: 1.0
To: Chris Faylor <cgf AT cygnus DOT com>
CC: cygdev <cygwin-developers AT sourceware DOT cygnus DOT com>
Subject: Re: access-patch
References: <387A450F DOT 6F2F272A AT vinschen DOT de> <20000110155112 DOT A17460 AT cygnus DOT com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

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.

Bye,
Corinna