This is the mail archive of the cygwin 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: Permissions change concern


On 4/18/2016 2:55 PM, Corinna Vinschen wrote:
On Apr 18 09:14, Eliot Moss wrote:
On 4/18/2016 6:18 AM, Corinna Vinschen wrote:
On Apr 16 22:17, Eliot Moss wrote:

Your example looks like 775.  But even then, the actual permissions
depend on the mode bits given to the open(2) call, which are usually 666
in this echo example.  The resulting perms should be 664.

Thank you -- I am convinced and agree, and that is what I get.

Cygwin also takes the umask value into account if the newly created
ACL only contains default POSIX permissions.  That's not quite correct
in terms of POSIX 1003.1e draft 17, but in contrast to Linux, Windows
directory ACLs practically always have default ACEs, so we have to
compromise a bit, or umask just has no effect at all.  I fixed a
problem with the permssion handling in case a mask value is present, but
that shouldn't matter in your case.

2) If the directory has g+s set (visible from -s- in the flags shown
    by getfacl), the directory's group is Cygwin, and my primary group is moss,
    then the file gets created with group moss, not group Cygwin (which is
    what g+s is supposed to mean, right?)

There was still another bug in the same code snippet which disallowed to
set the special POSIX bits S_ISVTX/S_ISGID/S_ISUID under some
circumstances, but apart from that what you describe works for me if the
bit is set in the parent dir and the parent dir inherits entries.

But this only works if you create the file with a Cygwin executable and
the parent dir has inheritable ACEs.

I uploaded another snapshot to https://cygwin.com/snapshots/ to make
sure the aforementioned bug can be tested as well.  Please check the
latest snapshot again.

However, if the s bit (aka S_ISGID) doesn't quite work as desired, I
have to defer any solution (if possible) until after my vaca.

The g+s bit now works as I hoped/expected. Thanks!     E

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple


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