Skip to content

Commit 5da6fdf

Browse files
committed
Cygwin: set_posix_access: Use DEF objects when creating default ACL entries from scratch
When creating default ACL entries from scratch, set_posix_access accidentally sets the type to GROUP_OBJ and OTHER_OBJ, rather than DEF_GROUP_OBJ/DEF_OTHER_OBJ. This shouldn't be noticable most of time since creating from scratch is seldom necessary. Fixes: bc444e5 ("Reapply POSIX ACL changes.") Signed-off-by: Corinna Vinschen <[email protected]> (cherry picked from commit aa481e0)
1 parent 39078c9 commit 5da6fdf

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

winsup/cygwin/sec/acl.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -219,10 +219,10 @@ set_posix_access (mode_t attr, uid_t uid, gid_t gid,
219219
aclbufp[3].a_type = DEF_USER_OBJ;
220220
aclbufp[3].a_id = ACL_UNDEFINED_ID;
221221
aclbufp[3].a_perm = (attr >> 6) & S_IRWXO;
222-
aclbufp[4].a_type = GROUP_OBJ;
222+
aclbufp[4].a_type = DEF_GROUP_OBJ;
223223
aclbufp[4].a_id = ACL_UNDEFINED_ID;
224224
aclbufp[4].a_perm = (attr >> 3) & S_IRWXO;
225-
aclbufp[5].a_type = OTHER_OBJ;
225+
aclbufp[5].a_type = DEF_OTHER_OBJ;
226226
aclbufp[5].a_id = ACL_UNDEFINED_ID;
227227
aclbufp[5].a_perm = attr & S_IRWXO;
228228
nentries += MIN_ACL_ENTRIES;

0 commit comments

Comments
 (0)