From c25504d1f9e2d02afb0f14093a0d16d6b6efb913 Mon Sep 17 00:00:00 2001
From: Matthew Swift <matthew.swift@forgerock.com>
Date: Fri, 10 Apr 2015 16:18:30 +0000
Subject: [PATCH] OPENDJ-1878: remove unnecessary entry read locks.
---
opendj-server-legacy/src/main/java/org/opends/server/controls/ProxiedAuthV2Control.java | 37 ++++++++++---------------------------
1 files changed, 10 insertions(+), 27 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/controls/ProxiedAuthV2Control.java b/opendj-server-legacy/src/main/java/org/opends/server/controls/ProxiedAuthV2Control.java
index a0505f5..f6c37ae 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/controls/ProxiedAuthV2Control.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/controls/ProxiedAuthV2Control.java
@@ -27,8 +27,6 @@
package org.opends.server.controls;
import java.io.IOException;
-import java.util.concurrent.locks.Lock;
-
import org.forgerock.i18n.LocalizableMessage;
import org.opends.server.api.AuthenticationPolicyState;
import org.opends.server.api.IdentityMapper;
@@ -236,35 +234,20 @@
authzDN = actualDN;
}
- final Lock entryLock = LockManager.lockRead(authzDN);
- if (entryLock == null)
+ Entry userEntry = DirectoryServer.getEntry(authzDN);
+ if (userEntry == null)
{
- throw new DirectoryException(ResultCode.BUSY,
- ERR_PROXYAUTH2_CANNOT_LOCK_USER.get(authzDN));
+ // The requested user does not exist.
+ LocalizableMessage message = ERR_PROXYAUTH2_NO_SUCH_USER.get(lowerAuthzID);
+ throw new DirectoryException(ResultCode.AUTHORIZATION_DENIED, message);
}
- try
- {
- Entry userEntry = DirectoryServer.getEntry(authzDN);
- if (userEntry == null)
- {
- // The requested user does not exist.
- LocalizableMessage message = ERR_PROXYAUTH2_NO_SUCH_USER.get(lowerAuthzID);
- throw new DirectoryException(ResultCode.AUTHORIZATION_DENIED,
- message);
- }
+ // FIXME -- We should provide some mechanism for enabling debug
+ // processing.
+ checkAccountIsUsable(userEntry);
- // FIXME -- We should provide some mechanism for enabling debug
- // processing.
- checkAccountIsUsable(userEntry);
-
- // If we've made it here, then the user is acceptable.
- return userEntry;
- }
- finally
- {
- LockManager.unlock(authzDN, entryLock);
- }
+ // If we've made it here, then the user is acceptable.
+ return userEntry;
}
}
else if (lowerAuthzID.startsWith("u:"))
--
Gitblit v1.10.0