Re: [PATCH] Ensure that the default ACL contains the standard entries

2011-03-16 Thread Christian Franke
Hi Corinna, Corinna Vinschen wrote: Hi Christian, On Dec 16 12:10, Corinna Vinschen wrote: - Fix acl(2) by handling deny ACEs at all. - Implement the POSIX 1003.1e functions (maybe simply in terms of the existing Solaris API). - Add missing Solaris ACL functions (acl_get, facl_get, a

Re: [PATCH] Ensure that the default ACL contains the standard entries

2011-03-11 Thread Corinna Vinschen
Hi Christian, On Dec 16 12:10, Corinna Vinschen wrote: > On Dec 15 19:46, Christian Franke wrote: > > BTW: Are there any long term plans to actually implement the acl "mask" ? > > Should be possible by mapping the "mask" restrictions to deny acl > > entries for each named entry: > > There are no

Re: [PATCH] Ensure that the default ACL contains the standard entries

2010-12-22 Thread Corinna Vinschen
On Dec 20 22:33, Christian Franke wrote: > Corinna Vinschen wrote: > >- Add Solaris NFSv4 ACLs, which, coincidentally, are almost equivalent > > to Windows ACLs. This would work nicely for NTFS ACLs, of course. > > See http://docs.sun.com/app/docs/doc/819-2252/acl-5?l=en&a=view > > Yes NFSv4

Re: [PATCH] Ensure that the default ACL contains the standard entries

2010-12-20 Thread Christian Franke
Hi Corinna, Corinna Vinschen wrote: Hi Christian, On Dec 15 19:46, Christian Franke wrote: ... BTW: Are there any long term plans to actually implement the acl "mask" ? Should be possible by mapping the "mask" restrictions to deny acl entries for each named entry: There are no such plan

Re: [PATCH] Ensure that the default ACL contains the standard entries

2010-12-16 Thread Corinna Vinschen
Hi Christian, On Dec 15 19:46, Christian Franke wrote: > Corinna Vinschen wrote: > >> > >>New patch attached. > >Thanks, applied. > > > > Thanks - rsync issue is now fixed. Good start. > >>mkdir() may duplicate Windows ACL entries. Testcase (German XP SP3): > >>[...] > >>Problem in security.cc:

Re: [PATCH] Ensure that the default ACL contains the standard entries

2010-12-15 Thread Christian Franke
Corinna Vinschen wrote: New patch attached. Thanks, applied. Thanks - rsync issue is now fixed. mkdir() may duplicate Windows ACL entries. Testcase (German XP SP3): [...] Problem in security.cc:alloc_sd() ? Indeed. Thanks for the report. I fixed that in CVS, hopefully.

Re: [PATCH] Ensure that the default ACL contains the standard entries

2010-12-15 Thread Corinna Vinschen
On Dec 14 22:22, Christian Franke wrote: > Hi Corinna, > > Corinna Vinschen wrote: > >Hi Christian, > > > >On Dec 10 23:05, Christian Franke wrote: > >>The ACL from Cygwin always contains the three (USER|GROUP|OTHER)_OBJ > >>entries. It might be existing practice elsewhere to return these > >>entr

Re: [PATCH] Ensure that the default ACL contains the standard entries

2010-12-14 Thread Christian Franke
Hi Corinna, Corinna Vinschen wrote: Hi Christian, On Dec 10 23:05, Christian Franke wrote: The ACL from Cygwin always contains the three (USER|GROUP|OTHER)_OBJ entries. It might be existing practice elsewhere to return these entries also in the default ACL. The attached patch adds these en

Re: [PATCH] Ensure that the default ACL contains the standard entries

2010-12-11 Thread Corinna Vinschen
Hi Christian, On Dec 10 23:05, Christian Franke wrote: > The ACL from Cygwin always contains the three (USER|GROUP|OTHER)_OBJ > entries. It might be existing practice elsewhere to return these > entries also in the default ACL. The attached patch adds these > entries with empty permissions if nece

[PATCH] Ensure that the default ACL contains the standard entries

2010-12-10 Thread Christian Franke
The ACL from Cygwin always contains the three (USER|GROUP|OTHER)_OBJ entries. It might be existing practice elsewhere to return these entries also in the default ACL. The attached patch adds these entries with empty permissions if necessary. The patch would fix this rsync issue: http://cygwin