From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Sun, 14 Aug 2011 22:15:24 +0000 (-0400)
Subject: Windows: cm_FindServerByUuid correct lock acquisition
X-Git-Tag: upstream/1.6.0^2~10
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=ee422deb20a54eeedbb0be954a846e6bdc5954bc;p=packages%2Fo%2Fopenafs.git

Windows: cm_FindServerByUuid correct lock acquisition

Obtain cm_serverLock when 'locked' is FALSE instead
of when TRUE.

Reviewed-on: http://gerrit.openafs.org/5268
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
(cherry picked from commit b97383c1e68a0b94baa8d05b2ab2531f96e7b63c)

Change-Id: I1fb188fa493554dce366cbc4b84a32528486829d
Reviewed-on: http://gerrit.openafs.org/5269
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
---

diff --git a/src/WINNT/afsd/cm_server.c b/src/WINNT/afsd/cm_server.c
index c915f2c6d..bfee6275c 100644
--- a/src/WINNT/afsd/cm_server.c
+++ b/src/WINNT/afsd/cm_server.c
@@ -912,7 +912,7 @@ cm_FindServerByUuid(afsUUID *serverUuid, int type, int locked)
 {
     cm_server_t *tsp;
 
-    if (locked)
+    if (!locked)
         lock_ObtainRead(&cm_serverLock);
 
     for (tsp = cm_allServersp; tsp; tsp = tsp->allNextp) {