From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Fri, 9 Sep 2011 16:47:52 +0000 (-0400)
Subject: Windows: ignore SYNCHRONIZE privilege in cm_CheckNTOpen
X-Git-Tag: upstream/1.6.1.pre1^2~109
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=0d698501d3446230e46fcfb646a3430424044daa;p=packages%2Fo%2Fopenafs.git

Windows: ignore SYNCHRONIZE privilege in cm_CheckNTOpen

SYNCHRONIZE has not meaning for AFS so just ignore it.  Do not
deny an open request because it is set.

Reviewed-on: http://gerrit.openafs.org/5410
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
(cherry picked from commit 793fdd3360982c48bf60eefbd9c56b4a3c1e09f0)

Change-Id: Ice8758dcbd0b355c9fdfb5af293b6d670ec8c1e4
Reviewed-on: http://gerrit.openafs.org/6037
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>
---

diff --git a/src/WINNT/afsd/cm_vnodeops.c b/src/WINNT/afsd/cm_vnodeops.c
index 904296313..c0c93b0e0 100644
--- a/src/WINNT/afsd/cm_vnodeops.c
+++ b/src/WINNT/afsd/cm_vnodeops.c
@@ -194,6 +194,9 @@ long cm_CheckNTOpen(cm_scache_t *scp, unsigned int desiredAccess,
     osi_assertx(ldpp != NULL, "null cm_lock_data_t");
     *ldpp = NULL;
 
+    /* Ignore the SYNCHRONIZE privilege */
+    desiredAccess &= ~SYNCHRONIZE;
+
     /* Always allow delete; the RPC will tell us if it's OK */
     rights = 0;