diff options
| author | Ken Brown <kbrown@cornell.edu> | 2019-08-26 13:38:31 -0400 |
|---|---|---|
| committer | Ken Brown <kbrown@cornell.edu> | 2019-08-26 13:38:31 -0400 |
| commit | 398476acd2dca8d68044e1b6e68eb41a16086f79 (patch) | |
| tree | ff7491e41d48def6f34b31890f3e74eeeb7c1717 | |
| parent | d0651704411d674a29a2af158388a22ef7355d53 (diff) | |
| download | cygnal-398476acd2dca8d68044e1b6e68eb41a16086f79.tar.gz cygnal-398476acd2dca8d68044e1b6e68eb41a16086f79.tar.bz2 cygnal-398476acd2dca8d68044e1b6e68eb41a16086f79.zip | |
Cygwin: get_posix_access: avoid negative subscript
Don't refer to lacl[pos] unless we know that pos >= 0.
| -rw-r--r-- | winsup/cygwin/sec_acl.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/winsup/cygwin/sec_acl.cc b/winsup/cygwin/sec_acl.cc index 933bfa69d..67749d7b1 100644 --- a/winsup/cygwin/sec_acl.cc +++ b/winsup/cygwin/sec_acl.cc @@ -807,9 +807,9 @@ get_posix_access (PSECURITY_DESCRIPTOR psd, lacl[pos].a_id = ACL_UNDEFINED_ID; lacl[pos].a_perm = CYG_ACE_MASK_TO_POSIX (ace->Mask); aclsid[pos] = well_known_null_sid; + has_class_perm = true; + class_perm = lacl[pos].a_perm; } - has_class_perm = true; - class_perm = lacl[pos].a_perm; } if (ace->Header.AceFlags & SUB_CONTAINERS_AND_OBJECTS_INHERIT) { @@ -820,9 +820,9 @@ get_posix_access (PSECURITY_DESCRIPTOR psd, lacl[pos].a_id = ACL_UNDEFINED_ID; lacl[pos].a_perm = CYG_ACE_MASK_TO_POSIX (ace->Mask); aclsid[pos] = well_known_null_sid; + has_def_class_perm = true; + def_class_perm = lacl[pos].a_perm; } - has_def_class_perm = true; - def_class_perm = lacl[pos].a_perm; } } } |