spring-projectsGH-8699: Atomic Redis script for unlock()

IllegalStateException should not be thrown from try block

Author: EddieChoCho

spring-integration-redis/src/main/java/org/springframework/integration/redis/util/RedisLockRegistry.java

@@ -476,13 +476,9 @@ public final void unlock() {
 
 		private void removeLockKey() {
 			if (RedisLockRegistry.this.unlinkAvailable) {
+				Boolean unlinkResult = null;
 				try {
-					boolean unlinkResult = removeLockKeyInnerUnlink();
-					if (!unlinkResult) {
-						throw new IllegalStateException("Lock was released in the store due to expiration. " +
-								"The integrity of data protected by this lock may have been compromised.");
-					}
-					return;
+					unlinkResult = removeLockKeyInnerUnlink();
 				}
 				catch (Exception ex) {
 					RedisLockRegistry.this.unlinkAvailable = false;
@@ -495,6 +491,14 @@ private void removeLockKey() {
 								"falling back to the regular DELETE command: " + ex.getMessage());
 					}
 				}
+
+				if(Boolean.TRUE.equals(unlinkResult)) {
+					return;
+				}
+				else if (Boolean.FALSE.equals(unlinkResult)) {
+					throw new IllegalStateException("Lock was released in the store due to expiration. " +
+							"The integrity of data protected by this lock may have been compromised.");
+				}
 			}
 			if (!removeLockKeyInnerDelete()) {
 				throw new IllegalStateException("Lock was released in the store due to expiration. " +