From b4ad7197fb1ad7dccdab927b93b52a5128cd293f Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Fri, 2 Mar 2012 17:07:17 +0000 Subject: * flock.cc (allow_others_to_sync): Use RtlGetDaclSecurityDescriptor rather than accessing the SECURITY_DESCRIPTOR structure directly. Take no DACL and NULL DACL into account. --- winsup/cygwin/flock.cc | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) (limited to 'winsup/cygwin/flock.cc') diff --git a/winsup/cygwin/flock.cc b/winsup/cygwin/flock.cc index a4d28e5f6..c66891a35 100644 --- a/winsup/cygwin/flock.cc +++ b/winsup/cygwin/flock.cc @@ -176,8 +176,22 @@ allow_others_to_sync () /* Create a valid dacl pointer and set its size to be as big as there's room in the temporary buffer. Note that the descriptor is in self-relative format. */ - dacl = (PACL) ((char *) sd + (uintptr_t) sd->Dacl); - dacl->AclSize = NT_MAX_PATH * sizeof (WCHAR) - ((char *) dacl - (char *) sd); + BOOLEAN present, defaulted; + RtlGetDaclSecurityDescriptor (sd, &present, &dacl, &defaulted); + if (dacl == NULL) /* Everyone has all access anyway */ + { + done = true; + return; + } + else if (!present) + { + dacl = (PACL) (sd + 1); + RtlCreateAcl (dacl, MAX_PROCESS_SD_SIZE - sizeof *sd, ACL_REVISION); + } + else + { + dacl->AclSize = MAX_PROCESS_SD_SIZE - ((PBYTE) dacl - (PBYTE) sd); + } /* Allow everyone to SYNCHRONIZE with this process. */ status = RtlAddAccessAllowedAce (dacl, ACL_REVISION, SYNCHRONIZE, well_known_world_sid); -- cgit v1.2.3