Fix a bug inside DistributedLock.extendExpiration():

Return false (as per API) if we don't hold the lock. Change-Id: Id3c314f6e374c92a73ce6f1aa4b6fbcef0a80697

Fix a bug inside DistributedLock.extendExpiration():
Return false (as per API) if we don't hold the lock. Change-Id: Id3c314f6e374c92a73ce6f1aa4b6fbcef0a80697
Pavlin Radoslavov · Gerrit Code Review
Commit 20ded699347d4b54dae3086bfa20ed3cb5e5a3e1 20ded699 1 parent 269670fc
Showing 1 changed file with 1 additions and 0 deletions
core/store/dist/src/main/java/org/onlab/onos/store/service/impl/DistributedLock.java
--- a/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/DistributedLock.java
View file @20ded69
+++ b/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/DistributedLock.java
View file @20ded69
@@ -132,6 +132,7 @@ public class DistributedLock implements Lock {
         if (!isLocked()) {
             log.warn("Ignoring request to extend expiration for lock {}."
                     + " ExtendExpiration must be called for locks that are already acquired.", path);
+             return false;
         }
 
         if (databaseService.putIfValueMatches(