This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: access()


On Thu, Feb 20, 2003 at 08:15:34PM -0500, Pierre A. Humblet wrote:
> However bash already uses access() when AFS is defined. Thus it
> would be a 1/2 line patch in bash (test.c and findcmd.c) to also
> use access() for Cygwin. 
> - #if defined (AFS)
> + #if defined (AFS) || defined (__CYGWIN__)
> That would be a significant improvement, IMO. What do you think?

Yes, I'll change that.  Thanks for the hint.

> 2003-02-21  Pierre Humblet  <pierre dot humblet at ieee dot org>
> 
> 	* autoload.cc (AccessCheck): Add.
> 	(DuplicateToken): Add.
> 	* security.h (check_file_access): Declare.
> 	* syscalls.cc (access): Convert path to Windows, check existence
> 	and readonly attribute. Call check_file_access instead of acl_access.
> 	* security.cc (check_file_access): Create.
> 	* sec_acl (acl_access): Delete.

I'm impressed.  Works nice with no more handcrafted messing around
with ACLs.

Applied.

Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin at cygwin dot com
Red Hat, Inc.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]