diff options
| author | Corinna Vinschen <corinna@vinschen.de> | 2001-08-07 20:14:59 +0400 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2001-08-07 20:14:59 +0400 |
| commit | c0ae23dc47de2d2ea8fa98b7cca386f0ccf24a07 (patch) | |
| tree | 325945d5177a34903918dde2d0e911ea4be3efb7 /winsup/cygwin | |
| parent | 86fb0393244e5827070e0bb0328a5b40fe5c0268 (diff) | |
* security.cc (alloc_sd): Don't set FILE_DELETE_CHILD for group
if S_ISVTX attribute is given.
* dir.cc (mkdir): Allow immediate setting of S_ISUID, S_ISGID and
S_ISVTX attribute.
* syscalls.cc (_open): Ditto.
Diffstat (limited to 'winsup/cygwin')
| -rw-r--r-- | winsup/cygwin/ChangeLog | 8 | ||||
| -rw-r--r-- | winsup/cygwin/dir.cc | 4 | ||||
| -rw-r--r-- | winsup/cygwin/security.cc | 3 | ||||
| -rw-r--r-- | winsup/cygwin/syscalls.cc | 2 |
4 files changed, 13 insertions, 4 deletions
diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index fc87d8e5f..5e8d12e42 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,11 @@ +Tue Aug 7 18:11:00 2001 Corinna Vinschen <corinna@vinschen.de> + + * security.cc (alloc_sd): Don't set FILE_DELETE_CHILD for group + if S_ISVTX attribute is given. + * dir.cc (mkdir): Allow immediate setting of S_ISUID, S_ISGID and + S_ISVTX attribute. + * syscalls.cc (_open): Ditto. + Tue Aug 7 16:24:00 2001 Corinna Vinschen <corinna@vinschen.de> * dir.cc (mkdir): Set security attributes correctly for diff --git a/winsup/cygwin/dir.cc b/winsup/cygwin/dir.cc index 022a8d663..e9628b377 100644 --- a/winsup/cygwin/dir.cc +++ b/winsup/cygwin/dir.cc @@ -320,14 +320,14 @@ mkdir (const char *dir, mode_t mode) goto done; if (allow_ntsec && real_dir.has_acls ()) - set_security_attribute (S_IFDIR | ((mode & 0777) & ~cygheap->umask), + set_security_attribute (S_IFDIR | ((mode & 07777) & ~cygheap->umask), &sa, alloca (256), 256); if (CreateDirectoryA (real_dir.get_win32 (), &sa)) { if (!allow_ntsec && allow_ntea) set_file_attribute (real_dir.has_acls (), real_dir.get_win32 (), - S_IFDIR | ((mode & 0777) & ~cygheap->umask)); + S_IFDIR | ((mode & 07777) & ~cygheap->umask)); res = 0; } else diff --git a/winsup/cygwin/security.cc b/winsup/cygwin/security.cc index d41d0a174..5a778e902 100644 --- a/winsup/cygwin/security.cc +++ b/winsup/cygwin/security.cc @@ -1393,7 +1393,8 @@ alloc_sd (uid_t uid, gid_t gid, const char *logsrv, int attribute, if (attribute & S_IXGRP) group_allow |= FILE_GENERIC_EXECUTE; if ((attribute & (S_IFDIR | S_IWGRP | S_IXGRP)) - == (S_IFDIR | S_IWGRP | S_IXGRP)) + == (S_IFDIR | S_IWGRP | S_IXGRP) + && !(attribute & S_ISVTX)) group_allow |= FILE_DELETE_CHILD; /* Construct allow attribute for everyone. */ diff --git a/winsup/cygwin/syscalls.cc b/winsup/cygwin/syscalls.cc index e7d6d73c9..a779c3ce4 100644 --- a/winsup/cygwin/syscalls.cc +++ b/winsup/cygwin/syscalls.cc @@ -466,7 +466,7 @@ _open (const char *unix_path, int flags, ...) set_errno (ENMFILE); else if ((fh = cygheap->fdtab.build_fhandler (fd, unix_path, NULL)) == NULL) res = -1; // errno already set - else if (!fh->open (unix_path, flags, (mode & 0777) & ~cygheap->umask)) + else if (!fh->open (unix_path, flags, (mode & 07777) & ~cygheap->umask)) { cygheap->fdtab.release (fd); res = -1; |