* flock.cc (allow_others_to_sync): Reorder conditional expression to

	check for DACL presence first, since otherwise dacl pointer value is
	undefined.  Add comment.

1 files changed, 6 insertions, 6 deletions

diff --git a/winsup/cygwin/flock.cc b/winsup/cygwin/flock.cc
index c66891a35..176e4869e 100644
--- a/winsup/cygwin/flock.cc
+++ b/winsup/cygwin/flock.cc
@@ -178,16 +178,16 @@ allow_others_to_sync ()
      is in self-relative format. */
   BOOLEAN present, defaulted;
   RtlGetDaclSecurityDescriptor (sd, &present, &dacl, &defaulted);
-  if (dacl == NULL) /* Everyone has all access anyway */
-    {
-      done = true;
-      return;
-    }
-  else if (!present)
+  if (!present) /* If so, dacl has undefined value. */
     {
       dacl = (PACL) (sd + 1);
       RtlCreateAcl (dacl, MAX_PROCESS_SD_SIZE - sizeof *sd, ACL_REVISION);
     }
+  else if (dacl == NULL) /* Everyone has all access anyway */
+    {
+      done = true;
+      return;
+    }
   else
     {
       dacl->AclSize = MAX_PROCESS_SD_SIZE - ((PBYTE) dacl - (PBYTE) sd);