access()

Corinna Vinschen cygwin-patches@cygwin.com
Fri Feb 21 14:32:00 GMT 2003


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.humblet@ieee.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@cygwin.com
Red Hat, Inc.



More information about the Cygwin-patches mailing list