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