Merge pull request #384 from dystedium/fixlock

Lock.release(): reorder code so assignment to self.acquired_until is protected by the lock.

1 files changed, 3 insertions, 2 deletions

diff --git a/redis/client.py b/redis/client.py
index 0c898bd..79a2a54 100644
--- a/redis/client.py
+++ b/redis/client.py
@@ -2207,6 +2207,7 @@ class Lock(object):
             raise ValueError("Cannot release an unlocked lock")
         existing = float(self.redis.get(self.name) or 1)
         # if the lock time is in the future, delete the lock
-        if existing >= self.acquired_until:
-            self.redis.delete(self.name)
+        delete_lock = existing >= self.acquired_until
         self.acquired_until = None
+        if delete_lock:
+            self.redis.delete(self.name)