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>

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

- Raw text -